User token stops working 8 hours after /github connect (classic OAuth App)
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Anfängerfreundlichkeit
- 52/100
- Issue-Typ
- Bug
- Klarheit
- Klar beschrieben
- Aktivitätsstatus
- Aktiv
- Tech-Stack
- github, go
- Bereich
- authentication, backend
Rechercherichtung
Start with the OAuth flow and token storage around server/plugin/plugin.go, then inspect server/plugin/graphql/client.go and the existing mm_34646_token_refresh.go reset path. Confirm whether GitHub returns expires_in and refresh_token for this OAuth flow, then reproduce the failure and identify the point where refresh handling and tests should be added. Done means expiring tokens are refreshed before the next GitHub call, revoked-token behavior remains covered, and relevant tests pass.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
On our self-hosted Mattermost, the token the plugin gets from /github connect stops working about 8 hours later. From then on every GitHub call returns 401 Bad credentials, and the next morning the daily reminder triggers the “Your Github account was disconnected due to an invalid or revoked authorization token” DM.
Environment
- Mattermost Team Edition 11.11.1 (since upgraded to 12.0.0)
- GitHub plugin 2.6.0
- A classic OAuth App owned by our organization, not a GitHub App. Its settings page has no token expiration option.
- One connected user, a bot account
Timing
| Connect (UTC) | First 401 (UTC) | Lifetime |
|---|---|---|
| 13 Oct 07:29:36 | 13 Oct 15:30:11 | 8h 00m |
| 14 Oct 07:58:04 | 14 Oct 16:15:51 | 8h 17m |
| 15 Oct 06:20:15 | 15 Oct 14:31:47 | 8h 11m |
| 16 Oct 08:33:43 | 16 Oct 16:35:11 | 8h 01m |
Connect times come from the account’s security log (oauth_access.create), 401 times from the plugin’s Failed to search for the sidebar data errors. The 401 appears at the first plugin call after the 8 hours, which explains the extra minutes.
What we ruled out
- The MM-34646 reset in
mm_34646_token_refresh.go. It skips users who connected through/github connect, since those getMM34646ResetTokenDone: true, and itsUpdated user access token for MM-34646debug line never appears in our logs. - Revocation. The security log has no
oauth_authorization.destroy, password change or 2FA change. It does have anoauth_access.regenerateby “GitHub System” within a second of everyoauth_access.create, but that also happens for an unrelated integration (Copilot), so it looks like part of normal token issuance. - Mattermost itself. No restart or plugin configuration change around the first failure.
Suggestion
Eight hours is the lifetime of GitHub’s expiring user tokens. If GitHub now issues those to OAuth Apps too, the plugin needs to keep the refresh token and refresh on expiry. Today it wraps the stored token in oauth2.StaticTokenSource (server/plugin/plugin.go, server/plugin/graphql/client.go), so it never refreshes. We inferred the 8 hours from the timings above and haven’t inspected the token response for expires_in or refresh_token. Can anyone confirm whether GitHub changed this for classic OAuth Apps?
- Vorherrschende Sprache
- Go
- Sterne
- 181
- Forks
- 181
- Ø Merge
- 22 Std. 10 Min.
- Gemergte PRs (30 T.)
- 5
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Keine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus mattermost/mattermost-plugin-github
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 58/100
-
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 55/100
-
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 48/100
mattermost/mattermost-plugin-github#996 · 1 Kommentar ·
-
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 35/100
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 35/100
mattermost/mattermost-plugin-github#937 · 2 Kommentare ·
Alle Issues in mattermost/mattermost-plugin-github
Ähnliche Issues
-
[Bug]: APIGW request ID headers are inconsistent between server and clientEvtl. vergeben @zouxingyuks hat das heute übernommen. Offenproduct/backlog
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 74/100
TencentBlueKing/bk-nodemgr#3782 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 91/100
Maintainer antworten meist innerhalb von 1 Tag
-
bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 84/100
praetorianer777/Dynamo#53 ·
Maintainer antworten meist innerhalb von 1 Tag
-
internal test setup
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 76/100
SAP/terraform-provider-scc#430 ·
-
internal test setup
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
SAP/terraform-provider-btp#1743 ·
Maintainer antworten meist innerhalb von 1 Tag