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.

Sign up for GitHub

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

Jump to bottom

incorrect url Oauth2 #6843

Closed
2 of 7 tasks
Aviortheking opened this issue May 4, 2019 · 0 comments · Fixed by #6894
Closed
2 of 7 tasks

incorrect url Oauth2 #6843

Aviortheking opened this issue May 4, 2019 · 0 comments · Fixed by #6894
Labels
topic/ui Change the appearance of the Gitea UI type/bug
Milestone
1.8.2

Comments

@Aviortheking
Copy link

  • Gitea version (or commit ref): 1.9.0+dev-191-g2933ae4e8
  • Git version: not relevant
  • Operating system: docker in linux
  • 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

The url in the @avior is not the one set in ROOT_URL (should be: git.delta-wings.net) config but the url passed by my proxy (nginx) (localhost:3002)

Screenshots

image
@lafriks lafriks added type/bug topic/ui Change the appearance of the Gitea UI labels May 4, 2019
@techknowlogick techknowlogick mentioned this issue May 9, 2019
Merged
@techknowlogick techknowlogick added this to the 1.9.0 milestone May 9, 2019
lafriks pushed a commit that referenced this issue May 9, 2019
@techknowlogick @lafriks
Use AppURL for Oauth user link (#6894)
2f21bc3 
* Use AppURL for Oauth user link

Fix #6843

* Update oauth.go

* Update oauth.go
@lafriks lafriks modified the milestones: 1.9.0, 1.8.2 May 9, 2019
zeripath pushed a commit to zeripath/gitea that referenced this issue May 11, 2019
@techknowlogick @zeripath
Use AppURL for Oauth user link (go-gitea#6894)
9bee091 
* Use AppURL for Oauth user link

Fix go-gitea#6843

* Update oauth.go

* Update oauth.go
techknowlogick pushed a commit that referenced this issue May 11, 2019
@zeripath @techknowlogick
Remove local clones & make hooks run on merge/edit/upload (#6672)
ce8de35 
* Add options to git.Clone to make it more capable

* Begin the process of removing the local copy and tidy up

* Remove Wiki LocalCopy Checkouts

* Remove the last LocalRepo helpers

* Remove WithTemporaryFile

* Enable push-hooks for these routes

* Ensure tests cope with hooks

Signed-off-by: Andrew Thornton <[email protected]>

* Remove Repository.LocalCopyPath()

* Move temporary repo to use the standard temporary path

* Fix the tests

Signed-off-by: Andrew Thornton <[email protected]>

* Remove LocalWikiPath

* Fix missing remove

Signed-off-by: Andrew Thornton <[email protected]>

* Use AppURL for Oauth user link (#6894)

* Use AppURL for Oauth user link

Fix #6843

* Update oauth.go

* Update oauth.go

* internal/ssh: ignore env command totally (#6825)

* ssh: ignore env command totally

* Remove commented code 

Needed fix described in issue #6889

* Escape the commit message on issues update and title in telegram hook (#6901)

* update sdk to latest (#6903)

* improve description of branch protection (fix #6886) (#6906)

The branch protection description text were not quite accurate.

* Fix logging documentation (#6904)

* ENABLE_MACARON_REDIRECT should be REDIRECT_MACARON_LOG

* Allow DISABLE_ROUTER_LOG to be set in the [log] section

* [skip ci] Updated translations via Crowdin

* Move sdk structs to modules/structs (#6905)

* move sdk structs to moduels/structs

* fix tests

* fix fmt

* fix swagger

* fix vendor
@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.
Assignees
No one assigned
Labels
topic/ui Change the appearance of the Gitea UI type/bug
Projects
None yet
Milestone
1.8.2
Development

Successfully merging a pull request may close this issue.

Use AppURL for Oauth user link
3 participants
@techknowlogick @lafriks @Aviortheking