Skip to content

Change to CHI misses some updates required for windows #14259

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

Closed
5 tasks
blazecoding2009 opened this issue Jan 5, 2021 · 1 comment · Fixed by #14263
Closed
5 tasks

Change to CHI misses some updates required for windows #14259

blazecoding2009 opened this issue Jan 5, 2021 · 1 comment · Fixed by #14263
Labels
skip-changelog This PR is irrelevant for the (next) changelog, for example bug fixes for unreleased features. type/bug
Milestone

Comments

@blazecoding2009
Copy link

blazecoding2009 commented Jan 5, 2021

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

Description

Got this error when I run go run main.go

# code.gitea.io/gitea/modules/auth/sso
modules\auth\sso\sspi_windows.go:69:27: undefined: ctx
modules\auth\sso\sspi_windows.go:79:56: undefined: ctx
modules\auth\sso\sspi_windows.go:82:37: undefined: ctx
modules\auth\sso\sspi_windows.go:87:15: undefined: ctx
modules\auth\sso\sspi_windows.go:88:3: undefined: ctx
modules\auth\sso\sspi_windows.go:89:3: undefined: ctx
modules\auth\sso\sspi_windows.go:90:3: undefined: ctx
modules\auth\sso\sspi_windows.go:94:37: undefined: ctx
modules\auth\sso\sspi_windows.go:113:24: too many arguments in call to s.newUser
modules\auth\sso\sspi_windows.go:113:25: undefined: ctx
modules\auth\sso\sspi_windows.go:113:24: too many errors```
## Screenshots

<!-- **If this issue involves the Web Interface, please include a screenshot** -->
@techknowlogick techknowlogick changed the title Issue from latest update Change to CHI misses some updates required for windows Jan 5, 2021
@techknowlogick
Copy link
Member

My recent PR #14260 won't resolve this, however it'll catch similar issues in the future, and it successfully replicates this issue which is a good test that the PR is good to have.

@techknowlogick techknowlogick added this to the 1.14.0 milestone Jan 5, 2021
@lunny lunny added the skip-changelog This PR is irrelevant for the (next) changelog, for example bug fixes for unreleased features. label Jan 6, 2021
@go-gitea go-gitea locked and limited conversation to collaborators Feb 11, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
skip-changelog This PR is irrelevant for the (next) changelog, for example bug fixes for unreleased features. type/bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants