This repository was archived by the owner on Sep 30, 2024. It is now read-only.
feat/enterpriseportal: all subscriptions APIs use enterprise portal DB #63959
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This was referenced Jul 19, 2024
Member
Author
bobheadxi
force-pushed
the
07-18-feat_enterpriseportal_subscriptions_read_apis_use_enterprise_portal_db
branch
from
7e86a43 to
152047a
Compare
bobheadxi
force-pushed
the
07-19-feat_enterpriseportal_all_subscriptions_apis_use_enterprise_portal_db
branch
from
fa7d5bc to
2ada484
Compare
bobheadxi
force-pushed
the
07-18-feat_enterpriseportal_subscriptions_read_apis_use_enterprise_portal_db
branch
from
152047a to
16bc317
Compare
bobheadxi
force-pushed
the
07-19-feat_enterpriseportal_all_subscriptions_apis_use_enterprise_portal_db
branch
from
2ada484 to
42751d4
Compare
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
Comment on lines
63
to
87
Check notice
Code scanning / Semgrep OSS
Semgrep Finding: security-semgrep-rules.semgrep-rules.generic.comment-tagging-rule
Comment on lines
611
to
757
Check notice
Code scanning / Semgrep OSS
Semgrep Finding: security-semgrep-rules.semgrep-rules.generic.comment-tagging-rule
bobheadxi
force-pushed
the
07-19-feat_enterpriseportal_all_subscriptions_apis_use_enterprise_portal_db
branch
from
42751d4 to
23df245
Compare
bobheadxi
force-pushed
the
07-18-feat_enterpriseportal_subscriptions_read_apis_use_enterprise_portal_db
branch
from
16bc317 to
d0bce4c
Compare
bobheadxi
force-pushed
the
07-19-feat_enterpriseportal_all_subscriptions_apis_use_enterprise_portal_db
branch
from
23df245 to
c2e0a5d
Compare
bobheadxi
force-pushed
the
07-18-feat_enterpriseportal_subscriptions_read_apis_use_enterprise_portal_db
branch
from
d0bce4c to
80c6ad1
Compare
bobheadxi
force-pushed
the
07-19-feat_enterpriseportal_all_subscriptions_apis_use_enterprise_portal_db
branch
from
c2e0a5d to
cca76b4
Compare
bobheadxi
force-pushed
the
07-18-feat_enterpriseportal_subscriptions_read_apis_use_enterprise_portal_db
branch
from
80c6ad1 to
99e6e2a
Compare
bobheadxi
force-pushed
the
07-19-feat_enterpriseportal_all_subscriptions_apis_use_enterprise_portal_db
branch
from
cca76b4 to
1cc2185
Compare
bobheadxi
force-pushed
the
07-18-feat_enterpriseportal_subscriptions_read_apis_use_enterprise_portal_db
branch
from
99e6e2a to
38aaf6b
Compare
bobheadxi
force-pushed
the
07-19-feat_enterpriseportal_all_subscriptions_apis_use_enterprise_portal_db
branch
from
1cc2185 to
c6041c4
Compare
bobheadxi
force-pushed
the
07-18-feat_enterpriseportal_subscriptions_read_apis_use_enterprise_portal_db
branch
from
38aaf6b to
d279b4e
Compare
bobheadxi
force-pushed
the
07-19-feat_enterpriseportal_all_subscriptions_apis_use_enterprise_portal_db
branch
from
c6041c4 to
aea1c9a
Compare
bobheadxi
force-pushed
the
07-26-feat_enterpriseportal_more_list_options_for_subscriptions_and_licenses
branch
from
480c94b to
8fd0533
Compare
bobheadxi
force-pushed
the
07-19-feat_enterpriseportal_all_subscriptions_apis_use_enterprise_portal_db
branch
from
06eecf1 to
2e737a6
Compare
Base automatically changed from
07-26-feat_enterpriseportal_more_list_options_for_subscriptions_and_licenses
to
main
August 9, 2024 16:43
bobheadxi
force-pushed
the
07-19-feat_enterpriseportal_all_subscriptions_apis_use_enterprise_portal_db
branch
from
2e737a6 to
b042f89
Compare
jac
approved these changes
Aug 9, 2024
Member
There was a problem hiding this comment.
Throughout this file it seems to use the string literal permission in numerous places rather than using the exported value from the sdk
Suggested change
| requiredScope := samsm2m.EnterprisePortalScope("subscription", scopes.ActionRead) | |
| requiredScope := samsm2m.EnterprisePortalScope(scopes.PermissionEnterprisePortalSubscription, scopes.ActionRead) |
Member
Author
There was a problem hiding this comment.
You're right, this predates the change to make the permissions public consts - updated, thank you!
bobheadxi
force-pushed
the
07-19-feat_enterpriseportal_all_subscriptions_apis_use_enterprise_portal_db
branch
from
b042f89 to
9ae24a6
Compare
2 tasks
Comment on lines
+411
to
+441
| // 🚨 SECURITY: Require appropriate M2M scope. | ||
| requiredScope := samsm2m.EnterprisePortalScope( | ||
| scopes.PermissionEnterprisePortalSubscription, scopes.ActionWrite) | ||
| clientAttrs, err := samsm2m.RequireScope(ctx, logger, s.store, requiredScope, req) | ||
| if err != nil { | ||
| return nil, err | ||
| } | ||
| logger = logger.With(clientAttrs...) | ||
|
|
||
| sub := req.Msg.GetSubscription() | ||
| if sub == nil { | ||
| return nil, connect.NewError(connect.CodeInvalidArgument, errors.New("subscription details are required")) | ||
| } | ||
|
|
||
| // Validate required arguments. | ||
| if strings.TrimSpace(sub.GetDisplayName()) == "" { | ||
| return nil, connect.NewError(connect.CodeInvalidArgument, errors.New("display_name is required")) | ||
| } | ||
|
|
||
| // Generate a new ID for the subscription. | ||
| if sub.Id != "" { | ||
| return nil, connect.NewError(connect.CodeInvalidArgument, errors.New("subscription_id can not be set")) | ||
| } | ||
| sub.Id, err = s.store.GenerateSubscriptionID() | ||
| if err != nil { | ||
| return nil, connectutil.InternalError(ctx, s.logger, err, "failed to generate new subscription ID") | ||
| } | ||
|
|
||
| // Check for an existing subscription, just in case. | ||
| if _, err := s.store.GetEnterpriseSubscription(ctx, sub.Id); err == nil { | ||
| return nil, connect.NewError(connect.CodeAlreadyExists, err) |
Check notice
Code scanning / Semgrep OSS
Semgrep Finding: security-semgrep-rules.semgrep-rules.generic.comment-tagging-rule
Comment on lines
+574
to
+604
| // 🚨 SECURITY: Require appropriate M2M scope. | ||
| requiredScope := samsm2m.EnterprisePortalScope( | ||
| scopes.PermissionEnterprisePortalSubscription, scopes.ActionWrite) | ||
| clientAttrs, err := samsm2m.RequireScope(ctx, logger, s.store, requiredScope, req) | ||
| if err != nil { | ||
| return nil, err | ||
| } | ||
| logger = logger.With(clientAttrs...) | ||
|
|
||
| subscriptionID := req.Msg.GetSubscriptionId() | ||
| if subscriptionID == "" { | ||
| return nil, connect.NewError(connect.CodeInvalidArgument, errors.New("subscription_id is required")) | ||
| } | ||
|
|
||
| if _, err := s.store.GetEnterpriseSubscription(ctx, subscriptionID); err != nil { | ||
| if errors.Is(err, subscriptions.ErrSubscriptionNotFound) { | ||
| return nil, connect.NewError(connect.CodeNotFound, err) | ||
| } | ||
| return nil, connectutil.InternalError(ctx, logger, err, "failed to find subscription") | ||
| } | ||
|
|
||
| archivedAt := s.store.Now() | ||
|
|
||
| // First, revoke all licenses associated with this subscription | ||
| licenses, err := s.store.ListEnterpriseSubscriptionLicenses(ctx, subscriptions.ListLicensesOpts{ | ||
| SubscriptionID: subscriptionID, | ||
| }) | ||
| if err != nil { | ||
| return nil, connectutil.InternalError(ctx, logger, err, "failed to list licenses for subscription") | ||
| } | ||
| revokedLicenses := make([]string, 0, len(licenses)) |
Check notice
Code scanning / Semgrep OSS
Semgrep Finding: security-semgrep-rules.semgrep-rules.generic.comment-tagging-rule
Comment on lines
+658
to
+687
| // 🚨 SECURITY: Require appropriate M2M scope. | ||
| requiredScope := samsm2m.EnterprisePortalScope( | ||
| scopes.PermissionEnterprisePortalSubscription, scopes.ActionWrite) | ||
| clientAttrs, err := samsm2m.RequireScope(ctx, logger, s.store, requiredScope, req) | ||
| if err != nil { | ||
| return nil, err | ||
| } | ||
| logger = logger.With(clientAttrs...) | ||
|
|
||
| create := req.Msg.GetLicense() | ||
| if create.GetId() != "" { | ||
| return nil, connect.NewError(connect.CodeInvalidArgument, errors.New("license.id cannot be set")) | ||
| } | ||
| subscriptionID := create.GetSubscriptionId() | ||
| if subscriptionID == "" { | ||
| return nil, connect.NewError(connect.CodeInvalidArgument, errors.New("license.subscription_id is required")) | ||
| } | ||
| sub, err := s.store.GetEnterpriseSubscription(ctx, subscriptionID) | ||
| if err != nil { | ||
| if errors.Is(err, subscriptions.ErrSubscriptionNotFound) { | ||
| return nil, connect.NewError(connect.CodeNotFound, err) | ||
| } | ||
| return nil, connectutil.InternalError(ctx, logger, err, "failed to find subscription") | ||
| } | ||
| if sub.ArchivedAt != nil { | ||
| return nil, connect.NewError(connect.CodeInvalidArgument, | ||
| errors.New("target subscription is archived")) | ||
| } | ||
|
|
||
| createdAt := s.store.Now() |
Check notice
Code scanning / Semgrep OSS
Semgrep Finding: security-semgrep-rules.semgrep-rules.generic.comment-tagging-rule
bobheadxi
deleted the
07-19-feat_enterpriseportal_all_subscriptions_apis_use_enterprise_portal_db
branch
This was referenced Aug 12, 2024
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This change follows https://github.com/sourcegraph/sourcegraph/pull/63858 by making the all subscriptions APIs read and write to the Enterprise Portal database, instead of dotcomdb, using the data that we sync from dotcomdb into Enterprise Portal.
With this PR, all initially proposed subscriptions APIs are at least partially implemented.
Uses hexops/valast#27 for custom
autogoldrendering ofutctime.TimeCloses https://linear.app/sourcegraph/issue/CORE-156
Part of https://linear.app/sourcegraph/issue/CORE-158
Test plan