Skip to content

Fix: sync mail login with OAuth authorization on load (glpi 10) - #161

Merged
RomainLvr merged 1 commit into
10.0/bugfixesfrom
support_46573
Oct 2, 2026
Merged

RomainLvr merged 1 commit into
10.0/bugfixesfrom
support_46573

Conversation

@Rom1-B

@Rom1-B Rom1-B commented Sep 28, 2026

Copy link
Copy Markdown
Contributor

Checklist before requesting a review

Please delete options that are not relevant.

  • I have performed a self-review of my code.
  • I have added tests (when available) that prove my fix is effective or that my feature works.
  • I have updated the CHANGELOG with a short functional description of the fix or new feature.
  • This change requires a documentation update.

Description

  • It fixes !46573
  • The mail collector login field for an OAuth IMAP connection is hidden and auto-filled from the selected OAuth authorization.
  • It was only refreshed when the user manually changed the authorization dropdown, never when the dropdown resolved its value on initial form load.
  • This left the login field empty in some cases, causing a misleading "cannot login, user or password wrong" error.
  • The login field is now also synced right after the authorization dropdown finishes loading.
  • No automated test was added: this repository has no PHPUnit or E2E test infrastructure, and the fix lives entirely in inline JS, so it was instead validated manually against a live GLPI instance.

Screenshots (if appropriate):

@Rom1-B
Rom1-B requested a review from RomainLvr September 28, 2026 07:44
@Rom1-B Rom1-B changed the title Fix: sync mail login with OAuth authorization on load Fix: sync mail login with OAuth authorization on load (glpi 10) Sep 28, 2026
@RomainLvr
RomainLvr merged commit 70e7f58 into 10.0/bugfixes Oct 2, 2026
7 checks passed
@RomainLvr
RomainLvr deleted the support_46573 branch October 2, 2026 07:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants