Skip to content

Commit

Permalink
Adding fixes for permission errors (403) (#925)
Browse files Browse the repository at this point in the history
* Adding fixes for permission errors

* Updating the warning message

* Adding fixes for too_many_requests (429) error

---------

Co-authored-by: KunalOfficial <[email protected]>
  • Loading branch information
2 people authored and kushalshit27 committed Sep 26, 2024
1 parent d394490 commit fb3e679
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 0 deletions.
5 changes: 5 additions & 0 deletions src/tools/auth0/handlers/connections.ts
Original file line number Diff line number Diff line change
Expand Up @@ -190,6 +190,11 @@ export default class ConnectionsHandler extends DefaultAPIHandler {
config: this.config,
});

// We add the expected changes to the scimHandler so it can track the progress of the SCIM changes.
// This is necessary because import / deploy actions are concurrent and we need to know when all updates are complete.
const { update, create, conflicts } = proposedChangesWithExcludedProperties;
this.scimHandler.expectedChanges = update.length + create.length + conflicts.length;

return proposedChangesWithExcludedProperties;
}

Expand Down
2 changes: 2 additions & 0 deletions src/tools/auth0/handlers/scimHandler.ts
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,8 @@ interface ScimScopes {
delete: boolean;
}

interface ScimScopes { read: boolean; create: boolean; update: boolean; delete: boolean }

/**
* The current version of this sdk use `node-auth0` v3. But `SCIM` features are not natively supported by v3.
* This is a workaround to make this SDK support SCIM without `node-auth0` upgrade.
Expand Down

0 comments on commit fb3e679

Please sign in to comment.