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

Improve error message when pinning when api is read-only #1319

Closed
olizilla opened this issue May 13, 2022 · 4 comments
Closed

Improve error message when pinning when api is read-only #1319

olizilla opened this issue May 13, 2022 · 4 comments
Assignees
Labels
kind/enhancement A net-new feature or improvement to an existing feature need/triage Needs initial labeling and prioritization pi/psa-follow-up topic/psa

Comments

@olizilla
Copy link
Contributor

olizilla commented May 13, 2022

We just drop a 503 service unavailable on the user. Can we give them a "API is read-only for maintenance. See https://status.web3.storage for details"

$ ipfs pin remote add --service=web3stage --name="ayy2.jpg" bafybeidluj5ub7okodgg5v6l4x3nytpivvcouuxgzuioa6vodg3xt2uqle                                                   
Error: reason: "", details: "": 503 Service Unavailable

When we try to create a new api token during a read-only period we get a better error like

{"code":"ERROR_MAINTENANCE","message":"API undergoing maintenance, check https://status.web3.storage/ for more info"}
@olizilla olizilla added kind/enhancement A net-new feature or improvement to an existing feature need/triage Needs initial labeling and prioritization labels May 13, 2022
@olizilla
Copy link
Contributor Author

Same issue when trying to list pins during a full "no reads or writes" maintenance mode

$ ipfs pin remote ls --service=web3stage                                                                                                                                 
Error: reason: "", details: "": 503 Service Unavailable

@joshghent
Copy link
Contributor

joshghent commented Aug 12, 2022

I believe this is resolved by #1710
I cannot recreate the issue locally now.
Please let me know if this is still occuring @olizilla

@flea89 flea89 assigned flea89 and unassigned joshghent Sep 6, 2022
@flea89
Copy link
Contributor

flea89 commented Sep 20, 2022

@flea89 to try to reproduce this one and close if not.

@mbommerez
Copy link

Closing ticket as issue couldn't be reproduced locally after fix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/enhancement A net-new feature or improvement to an existing feature need/triage Needs initial labeling and prioritization pi/psa-follow-up topic/psa
Projects
None yet
Development

No branches or pull requests

5 participants