Skip to content

fix: say the gateway mint limit is weekly and account-wide #1252

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.

Sign up for GitHub

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

Draft
gewenyu99 wants to merge 1 commit into
base: main
Choose a base branch
from
+3 −3
Draft

fix: say the gateway mint limit is weekly and account-wide #1252

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion src/lib/__tests__/gateway-session.test.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -321,7 +321,7 @@ describe('gatewayAuth', () => {
});

it.each([
[429, 'daily run limit'],
[429, 'weekly wizard run limit'],
[400, 'exactly one project'],
[403, 'access to this project'],
])('refuses the run on HTTP %i', async (status, fragment) => {
Expand Down
4 changes: 2 additions & 2 deletions src/lib/gateway-session.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -276,7 +276,7 @@ export class GatewayMintFailed extends WizardError {

/**
* Whether a mint status means "refused this run" rather than "not available".
* 429 the daily run limit, 403 revoked project access, 400 a login covering
* 429 the weekly run limit, 403 revoked project access, 400 a login covering
* more than one project, 401 a credential the mint does not accept, 404 an
* instance without the mint endpoint.
*/
Expand Down Expand Up @@ -342,7 +342,7 @@ function mintRefusalMessage(status: number, detail?: string): string {
if (detail) return detail;
switch (status) {
case 429:
return 'This wizard program has used its daily run limit. Try again tomorrow.';
return 'This account has used its weekly wizard run limit. Try again next week.';
case 403:
return 'Your access to this project has changed. Re-authenticate and try again.';
case 400:
Expand Down
Loading