-
-
Notifications
You must be signed in to change notification settings - Fork 5.4k
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
allow U2F with default settings for gitea in subpath #12990
Conversation
Codecov Report
@@ Coverage Diff @@
## master #12990 +/- ##
==========================================
- Coverage 42.59% 42.58% -0.01%
==========================================
Files 671 671
Lines 73625 73625
==========================================
- Hits 31363 31356 -7
- Misses 37183 37195 +12
+ Partials 5079 5074 -5
Continue to review full report at Codecov.
|
Blocking this as it needs more testing. |
* Refactor use TrimSuffix instead of TrimRight * TrimRight right * has #12990
Looking at the settings code it is definitely the case that the AppSubURL does not have a terminal "/" and it is always the case the the AppURL has a terminal slash - therefore the proposed PR is correctly stripping off the AppSubURL |
* allow U2F with default settings for gitea in subpath * use trim suffix Co-authored-by: zeripath <[email protected]>
* allow U2F with default settings for gitea in subpath * use trim suffix Co-authored-by: zeripath <[email protected]> Co-authored-by: zeripath <[email protected]>
credit to @zeripath for logic
Fix #10231
Likely fixes #5143