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

Users Never Signed-In, yet have repos published #10374

Closed
2 of 7 tasks
sefsh opened this issue Feb 20, 2020 · 10 comments
Closed
2 of 7 tasks

Users Never Signed-In, yet have repos published #10374

sefsh opened this issue Feb 20, 2020 · 10 comments
Labels
issue/confirmed Issue has been reviewed and confirmed to be present or accepted to be implemented type/bug

Comments

@sefsh
Copy link

sefsh commented Feb 20, 2020

  • Gitea version (or commit ref):
  • Git version: 1.11.1
  • Operating system: ubuntu:bionic-20180821
  • 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:
    N/A

Description

When I check for users with the Last Sign-In status Never Signed-In, I find users that have published repos on the platform. This shouldn't be possible.

Screenshots

bild

@jolheiser
Copy link
Member

Is there a chance someone transferred a repo to this account?
Or that the user created the repo via push-create if enabled before logging in?

@lunny
Copy link
Member

lunny commented Feb 21, 2020

There is a known issue that if user only push/pull via git, the Last Sign-In didn't update.

@rnowak
Copy link
Contributor

rnowak commented Feb 22, 2020

Chiming in to add that this is also the case if the user authenticates via a header set in REVERSE_PROXY_AUTHENTICATION_USER both for the web interface as well as git interactions over https.

2020-02-22 14_26_46-User Accounts

@lunny lunny added the type/bug label Feb 23, 2020
@stale
Copy link

stale bot commented Apr 23, 2020

This issue has been automatically marked as stale because it has not had recent activity. I am here to help clear issues left open even if solved or waiting for more insight. This issue will be closed if no further activity occurs during the next 2 weeks. If the issue is still valid just add a comment to keep it alive. Thank you for your contributions.

@stale stale bot added the issue/stale label Apr 23, 2020
@zeripath zeripath added the issue/confirmed Issue has been reviewed and confirmed to be present or accepted to be implemented label May 6, 2020
@stale stale bot removed the issue/stale label May 6, 2020
@fnetX
Copy link
Contributor

fnetX commented Apr 7, 2021

AFAICT this is also the case if a user registers and does actions within pers first session (this also explains how a user could create a repo which is not possible with push / pull alone).

@zeripath
Copy link
Contributor

zeripath commented Apr 8, 2021

Ok that's a much simpler bug to fix!

@furai
Copy link

furai commented Apr 10, 2021

I get the same issue - I have gitea behind reverse proxy and last sign-in doesn't update.
(Also for some reason when I log-in, I get presented with a blank screen until I refresh, and I don't know why.)

@zeripath
Copy link
Contributor

zeripath commented Apr 10, 2021

AFAICT this is also the case if a user registers and does actions within pers first session (this also explains how a user could create a repo which is not possible with push / pull alone).

I think that that this has been patrially fixed already in 1.14+ probably earlier (excepting potentially ReverseProxy)

SetLastLogin() does not appear to be called on API logins. Is it possible therefore that these are creations from the API?

@zeripath
Copy link
Contributor

I get the same issue - I have gitea behind reverse proxy and last sign-in doesn't update.
(Also for some reason when I log-in, I get presented with a blank screen until I refresh, and I don't know why.)

This should be fixed by #15304

@wolfogre
Copy link
Member

wolfogre commented Nov 9, 2022

AFAICT this is also the case if a user registers and does actions within pers first session (this also explains how a user could create a repo which is not possible with push / pull alone).

The case should be fixed by #21731.

@go-gitea go-gitea locked and limited conversation to collaborators May 3, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
issue/confirmed Issue has been reviewed and confirmed to be present or accepted to be implemented type/bug
Projects
None yet
Development

No branches or pull requests

9 participants