Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(auth)!: authentication and users #4438

Merged
merged 127 commits into from
Sep 26, 2024
Merged

feat(auth)!: authentication and users #4438

merged 127 commits into from
Sep 26, 2024

Conversation

mikeldking
Copy link
Contributor

No description provided.

@mikeldking mikeldking added the feature branch a feature branch that consolidates multiple features into a single commit on main label Aug 29, 2024
Copy link
Contributor

@axiomofjoy axiomofjoy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

auth feature branch

@axiomofjoy axiomofjoy changed the title feat(auth)authentication and users feat(auth)!: authentication and users Aug 29, 2024
@RogerHYang RogerHYang force-pushed the auth branch 2 times, most recently from 03a5ad0 to dc21f70 Compare September 4, 2024 15:27
mikeldking and others added 25 commits September 20, 2024 19:20
* Spike out serverside rate limiting middleware

* Refactor to keep multiple active partitions

* Create StrawberryRateLimiter extension

* Add rate limiter tests

* Update extension to run synchronously

* Ensure rate limiter extension works for both sync and async resolvers

* Clean up type annotations

* Improve tests and reset behavior

* Clarify edge cases in testing and documentation

* Use pop instead of `del`

* Remove extension implementation
* feat(auth): user of a given key

* remove user role not used

* format

* fix casing
* Add CreateUserApiKey mutation

* Build gql schema

* Clean up type annotations

* Remove needless imports
* fix user

* fix

* simplify

* fix lint error

---------

Co-authored-by: Mikyo King <mikyo@arize.com>
* viewer node

* feat(auth): rudamentary viewer context and profile page
* feat(auth): auth rbac components

* fix deps arrays
* Add API keys to viewer node

* Implement api_keys as a resolver on the user node
* feat(auth): add delete user api keys mutation

* add permission classes

* remove codegen

* code gen

* update generated
Co-authored-by: Dustin Ngo <dustin@arize.com>
…4500)

* Modify conftest to allow a threshold of flaky postgres tests

* Less permissive xfail limit

* xfail all database tests

* Make tests more permissive, only xfail when running all db tests
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

RogerHYang and others added 10 commits September 25, 2024 18:10
* fix(playwright): revert back to localhost for playwright

* fix
* fix: overflow of UI issues

* fix more
* Simplify experiment tests

* Remove breakpoint :(

* Reintroduce plateform skips

* remove pydantic-specific versions

* remove pydantic-specific versions

* Sleep a while

* Properly check score

* Fewer evaluators for consistency

* Remove needless empty dataset check

* Uncomment flaky skip

* Remove flaky assertion

---------

Co-authored-by: Alexander Song <axiomofjoy@gmail.com>
…4760)

* chore(ci): add pnpm cache and frozen-lockfile install for playwright

* try pnpm cache in pnpm step

* add back node

* test

* test

* test

* test
* chore(ci): move pnpm action above node

* remove null checks on username

* Remove flaky assertions

---------

Co-authored-by: Dustin Ngo <dustin@arize.com>
* fixdeprecate support for 3.8

* more changes
@mikeldking mikeldking marked this pull request as ready for review September 26, 2024 18:34
@dosubot dosubot bot added the size:XXL This PR changes 1000+ lines, ignoring generated files. label Sep 26, 2024
Copy link
Contributor

@anticorrelator anticorrelator left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👁️ 👄 👁️

@mikeldking mikeldking merged commit d394688 into main Sep 26, 2024
24 of 32 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature branch a feature branch that consolidates multiple features into a single commit on main size:XXL This PR changes 1000+ lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants