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

URL Composition Problems with Alternative Port Configurations #2403

Closed
3 of 7 tasks
smeddinck opened this issue Aug 26, 2017 · 10 comments
Closed
3 of 7 tasks

URL Composition Problems with Alternative Port Configurations #2403

smeddinck opened this issue Aug 26, 2017 · 10 comments
Labels
Milestone

Comments

@smeddinck
Copy link

  • Gitea version (or commit ref): 1.2
  • Git version:
  • Operating system: Debian 8
  • Database (use [x]):
    • PostgreSQL
    • MySQL
    • MSSQL
    • SQLite
  • Can you reproduce the bug at https://try.gitea.io:
    • Yes (provide example URL)
    • No
    • Not relevant
  • Log gist:

Description

I use an alternative port for gitea via ssl (on nginx). Many API related links (e.g. https://mysite.com/swagger.v1.json or slack integration backlinks) miss the port (since it should be https://mysite.com:4242/swagger.v1.json). Gitea 1.2 on Debian 8 with MySQL

@lunny lunny added the type/bug label Aug 26, 2017
@0rzech
Copy link
Contributor

0rzech commented Nov 26, 2017

It's the same for https://mysite.com/mypath/swagger.v1.json. It looks like ROOT_URL is not being prepended to Swagger's basePath. Unfortunately this makes Gitea's Swagger documentation unusable with custom root URLs.

@igsol
Copy link

igsol commented Aug 6, 2018

I have a similar problem. My Gitea server runs behind SSL reverse proxy which makes not default URL to Gitea service, e.g. https://example.com/**gitea**/.
But Swagger page continues to use https://example.com/api/v1/ with missed "gitea" part after hostname.

@0rzech
Copy link
Contributor

0rzech commented Aug 7, 2018

@igsol This should be fixed in #3572.

@lunny lunny closed this as completed Aug 8, 2018
@lafriks lafriks added this to the 1.6.0 milestone Aug 8, 2018
@0rzech
Copy link
Contributor

0rzech commented Aug 8, 2018

@lunny Mine and @igsol's case is solved, but I am not sure about @smeddinck's, as it seems to be a different kind of bug with different solution.

@igsol
Copy link

igsol commented Aug 8, 2018

@0rzech Thanks! I am on v1.4.3 still. Waiting for next release :)

@0rzech
Copy link
Contributor

0rzech commented Aug 8, 2018

@igsol You're welcome. ;)

@lunny
Copy link
Member

lunny commented Aug 8, 2018

@smeddinck could you confirm #3572 resolved your issue?

@0rzech
Copy link
Contributor

0rzech commented Aug 8, 2018

@igsol I forgot to remind you the fix will be available in 1.6 version, not in 1.5.

@igsol
Copy link

igsol commented Aug 8, 2018

@0rzech NP, it's not urgent. The bug actually is not so critical as for me, cause it makes Swagger UI usage a bit harder, but API itself works flawlessly :)

@0rzech
Copy link
Contributor

0rzech commented Aug 8, 2018

@igsol Sure thing! Just wanted to avoid potential confusion. ;)

@go-gitea go-gitea locked and limited conversation to collaborators Nov 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

5 participants