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: Implement internal Access API services #1932

Merged
merged 1 commit into from
Dec 10, 2024

Conversation

SanjayVas
Copy link
Member

No description provided.

@wfa-reviewable
Copy link

This change is Reviewable

@SanjayVas SanjayVas force-pushed the sanjayvas-access-api-internal-impl branch from 1902d74 to 0e02126 Compare November 21, 2024 20:18
@SanjayVas SanjayVas force-pushed the sanjayvas-access-api-db branch from 5cc9c81 to a51beb5 Compare November 21, 2024 20:18
@SanjayVas SanjayVas force-pushed the sanjayvas-access-api-internal-impl branch from 0e02126 to a3256d6 Compare November 25, 2024 19:35
@SanjayVas SanjayVas force-pushed the sanjayvas-access-api-db branch 2 times, most recently from c688e62 to 8ae8c7d Compare November 26, 2024 18:49
@SanjayVas SanjayVas force-pushed the sanjayvas-access-api-internal-impl branch from a3256d6 to 819f546 Compare November 26, 2024 18:50
@SanjayVas SanjayVas force-pushed the sanjayvas-access-api-db branch from 8ae8c7d to 4624c54 Compare November 26, 2024 19:16
@SanjayVas SanjayVas force-pushed the sanjayvas-access-api-internal-impl branch 2 times, most recently from a593803 to a6daa72 Compare November 26, 2024 19:18
@SanjayVas SanjayVas force-pushed the sanjayvas-access-api-db branch from 4624c54 to 7ebb4bc Compare November 26, 2024 19:18
@SanjayVas SanjayVas force-pushed the sanjayvas-access-api-internal-impl branch from a6daa72 to 98048db Compare November 27, 2024 19:25
@SanjayVas SanjayVas force-pushed the sanjayvas-access-api-db branch from 7ebb4bc to c4a3f56 Compare November 27, 2024 19:25
@SanjayVas SanjayVas force-pushed the sanjayvas-access-api-internal-impl branch from 98048db to 25e34e7 Compare November 27, 2024 21:37
@SanjayVas SanjayVas force-pushed the sanjayvas-access-api-db branch from c4a3f56 to 5778515 Compare November 27, 2024 21:37
@SanjayVas SanjayVas force-pushed the sanjayvas-access-api-internal-impl branch from 25e34e7 to 5b918af Compare November 27, 2024 23:42
@SanjayVas SanjayVas force-pushed the sanjayvas-access-api-db branch 2 times, most recently from 32c4306 to 644cfcd Compare December 3, 2024 21:17
@SanjayVas SanjayVas force-pushed the sanjayvas-access-api-internal-impl branch from 5b918af to ee2e12d Compare December 3, 2024 21:17
@SanjayVas SanjayVas force-pushed the sanjayvas-access-api-db branch from 644cfcd to cc777ea Compare December 3, 2024 21:22
@SanjayVas SanjayVas force-pushed the sanjayvas-access-api-internal-impl branch 2 times, most recently from 9281a3a to 42d2ead Compare December 4, 2024 00:32
@SanjayVas SanjayVas force-pushed the sanjayvas-access-api-db branch from cc777ea to f8e7efa Compare December 4, 2024 00:32
@SanjayVas SanjayVas force-pushed the sanjayvas-access-api-internal-impl branch from 42d2ead to 15e53bf Compare December 4, 2024 00:42
@SanjayVas SanjayVas force-pushed the sanjayvas-access-api-db branch from f8e7efa to e394e7d Compare December 4, 2024 00:42
@SanjayVas SanjayVas force-pushed the sanjayvas-access-api-internal-impl branch from 15e53bf to c579af8 Compare December 4, 2024 01:04
@SanjayVas SanjayVas force-pushed the sanjayvas-access-api-db branch 2 times, most recently from 9bf8694 to bb9195b Compare December 5, 2024 19:47
@SanjayVas SanjayVas force-pushed the sanjayvas-access-api-internal-impl branch from c579af8 to 4bd95d7 Compare December 5, 2024 19:47
@SanjayVas SanjayVas marked this pull request as ready for review December 6, 2024 00:56
@SanjayVas
Copy link
Member Author

SanjayVas commented Dec 6, 2024

Depends on #1900.

See design doc for more info, e.g. on how TLS clients are handled.

Copy link
Collaborator

@stevenwarejones stevenwarejones left a comment

Choose a reason for hiding this comment

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

Reviewed 25 of 42 files at r1, 18 of 18 files at r2, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @SanjayVas)

@SanjayVas SanjayVas force-pushed the sanjayvas-access-api-internal-impl branch from 4bd95d7 to e576c9d Compare December 7, 2024 02:37
@SanjayVas SanjayVas force-pushed the sanjayvas-access-api-db branch 2 times, most recently from e406690 to f4b2821 Compare December 7, 2024 02:42
@SanjayVas SanjayVas force-pushed the sanjayvas-access-api-internal-impl branch 2 times, most recently from acfed3a to 981fb32 Compare December 7, 2024 02:48
Base automatically changed from sanjayvas-access-api-db to main December 7, 2024 02:49
@SanjayVas SanjayVas force-pushed the sanjayvas-access-api-internal-impl branch from 981fb32 to d8a94f8 Compare December 9, 2024 20:17
@SanjayVas SanjayVas enabled auto-merge (squash) December 9, 2024 20:18
@SanjayVas SanjayVas force-pushed the sanjayvas-access-api-internal-impl branch from d8a94f8 to f9f9e70 Compare December 9, 2024 22:25
Copy link
Member Author

@SanjayVas SanjayVas left a comment

Choose a reason for hiding this comment

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

Reviewed 1 of 2 files at r3.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @stevenwarejones)

@SanjayVas SanjayVas force-pushed the sanjayvas-access-api-internal-impl branch from f9f9e70 to 4de6133 Compare December 10, 2024 21:08
Copy link
Member Author

@SanjayVas SanjayVas left a comment

Choose a reason for hiding this comment

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

Reviewed 1 of 1 files at r4.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @stevenwarejones)

@SanjayVas SanjayVas merged commit 2dc095b into main Dec 10, 2024
4 checks passed
@SanjayVas SanjayVas deleted the sanjayvas-access-api-internal-impl branch December 10, 2024 21:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants