-
Notifications
You must be signed in to change notification settings - Fork 27
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
🐛 OSPARC_API_BASE_URL variable ends with /
#7042
🐛 OSPARC_API_BASE_URL variable ends with /
#7042
Conversation
What about changing the DIRECTOR_V2_PUBLIC_API_BASE_URL: Annotated[AnyHttpUrl | str, AfterValidator(lambda v: str(v).rstrip("/")), Field(
...,
description="Base URL used to access the public api e.g. http://127.0.0.1:6000 for development or https://api.osparc.io",
)] |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #7042 +/- ##
===========================================
- Coverage 82.33% 69.12% -13.21%
===========================================
Files 1465 662 -803
Lines 57456 32207 -25249
Branches 1295 262 -1033
===========================================
- Hits 47304 22262 -25042
+ Misses 9965 9885 -80
+ Partials 187 60 -127
Continue to review full report in Codecov by Sentry.
|
fa6f0fa
to
ceb9150
Compare
Quality Gate passedIssues Measures |
What do these changes do?
strip ending
/
from URLRelated issue/s
How to test
Dev-ops checklist