search

jina-ai / jcloud

Simplify deploying and managing Jina projects on Jina Cloud
https://docs.jina.ai/concepts/jcloud/
Apache License 2.0
298 stars 13 forks source link

fix: when token is expired, prompt should show #56

Closed zac-li closed 2 years ago

zac-li commented 2 years ago

Goal This is for https://github.com/jina-ai/wolf/issues/143. When token is expired, it returns HTTPStatus.FORBIDDEN, we should capture that and show it to user.

Integration tests: https://github.com/jina-ai/jcloud/actions/runs/2472532350

@jina-ai/team-wolf

codecov[bot] commented 2 years ago

Codecov Report

Merging #56 (39e82a0) into main (882f625) will decrease coverage by 12.70%. The diff coverage is 0.00%.

@@             Coverage Diff             @@
##             main      #56       +/-   ##
===========================================
- Coverage   51.33%   38.62%   -12.71%     
===========================================
  Files          15       15               
  Lines         974      976        +2     
===========================================
- Hits          500      377      -123     
- Misses        474      599      +125
Flag Coverage Δ
jcloud 38.62% <0.00%> (-12.71%) :arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
jcloud/flow.py 34.73% <0.00%> (-54.89%) :arrow_down:
jcloud/helper.py 59.04% <0.00%> (-17.15%) :arrow_down:
jcloud/env_helper.py 84.67% <0.00%> (+10.48%) :arrow_up:
jcloud/api.py 45.58% <0.00%> (+17.64%) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 882f625...39e82a0. Read the comment docs.