diff --git a/web_src/js/features/common-global.js b/web_src/js/features/common-global.js index 65eb237dde..80916b049d 100644 --- a/web_src/js/features/common-global.js +++ b/web_src/js/features/common-global.js @@ -453,3 +453,11 @@ export function checkAppUrl() { showGlobalErrorMessage(`Your ROOT_URL in app.ini is "${appUrl}", it's unlikely matching the site you are visiting. Mismatched ROOT_URL config causes wrong URL links for web UI/mail content/webhook notification/OAuth2 sign-in.`, 'warning'); } + +export function checkAppUrlScheme() { + const curUrl = window.location.href; + // some users visit "http://domain" while appUrl is "https://domain", COOKIE_SECURE makes it impossible to sign in + if (curUrl.startsWith('http:') && appUrl.startsWith('https:')) { + showGlobalErrorMessage(`This instance is configured to run under HTTPS (by ROOT_URL config), you are accessing by HTTP. Mismatched scheme might cause problems for sign-in/sign-up.`, 'warning'); + } +} diff --git a/web_src/js/features/user-auth.js b/web_src/js/features/user-auth.js index a871ac471c..13147ca727 100644 --- a/web_src/js/features/user-auth.js +++ b/web_src/js/features/user-auth.js @@ -1,4 +1,9 @@ -import {checkAppUrl} from './common-global.js'; +import {checkAppUrl, checkAppUrlScheme} from './common-global.js'; + +export function initUserCheckAppUrl() { + if (!document.querySelector('.page-content.user.signin, .page-content.user.signup, .page-content.user.link-account')) return; + checkAppUrlScheme(); +} export function initUserAuthOauth2() { const outer = document.getElementById('oauth2-login-navigator'); diff --git a/web_src/js/index.js b/web_src/js/index.js index 4c3852b406..8cbeeea5d7 100644 --- a/web_src/js/index.js +++ b/web_src/js/index.js @@ -23,7 +23,7 @@ import {initFindFileInRepo} from './features/repo-findfile.js'; import {initCommentContent, initMarkupContent} from './markup/content.js'; import {initPdfViewer} from './render/pdf.js'; -import {initUserAuthOauth2} from './features/user-auth.js'; +import {initUserAuthOauth2, initUserCheckAppUrl} from './features/user-auth.js'; import { initRepoIssueDue, initRepoIssueReferenceRepositorySearch, @@ -184,6 +184,7 @@ onDomReady(() => { initCommitStatuses(); initCaptcha(); + initUserCheckAppUrl(); initUserAuthOauth2(); initUserAuthWebAuthn(); initUserAuthWebAuthnRegister();