') + ')', 'gi'); if (regex.test(text)) { found = true; var frag = document.createDocumentFragment(); var parts = text.split(regex); parts.forEach(function(part, i) { if (i % 2 === 0) { frag.appendChild(document.createTextNode(part)); } else { var span = document.createElement('span'); span.className = 'userscript-highlight'; span.textContent = part; frag.appendChild(span); } }); node.parentNode.replaceChild(frag, node); } }); } else if (node.nodeType === 1 && node.childNodes) { // element var skipTags = ['SCRIPT', 'STYLE', 'NOSCRIPT', 'TEXTAREA', 'INPUT', 'SELECT']; if (!skipTags.includes(node.tagName)) { Array.from(node.childNodes).forEach(highlight); } } } highlight(document.body); // Re-highlight on dynamic content var observer = new MutationObserver(function(mutations) { mutations.forEach(function(m) { m.addedNodes.forEach(function(node) { if (node.nodeType === 1 || node.nodeType === 3) highlight(node); }); }); }); observer.observe(document.body, { childList: true, subtree: true }); })(); } } catch(__e) { console.warn('[Userscript:Highlight Search Terms]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + ', 'i'); if (__m === '*' || __re.test(location.href)) { // Strip utm_, fbclid, gclid, etc. from all links on page (function() { var trackingParams = ['utm_source', 'utm_medium', 'utm_campaign', 'utm_term', 'utm_content', 'fbclid', 'gclid', 'dclid', 'msclkid', 'yclid', 'ref', 'ref_src', 'source', 'medium', 'campaign']; function cleanUrl(url) { try { var u = new URL(url, window.location.origin); var changed = false; trackingParams.forEach(function(p) { if (u.searchParams.has(p)) { u.searchParams.delete(p); changed = true; } }); return changed ? u.toString() : url; } catch (e) { return url; } } function cleanLinks() { document.querySelectorAll('a[href]').forEach(function(a) { var clean = cleanUrl(a.href); if (clean !== a.href) a.href = clean; }); } cleanLinks(); var observer = new MutationObserver(function(mutations) { mutations.forEach(function(m) { m.addedNodes.forEach(function(node) { if (node.nodeType === 1) { if (node.tagName === 'A') cleanLinks(); node.querySelectorAll('a[href]').forEach(function(a) { var clean = cleanUrl(a.href); if (clean !== a.href) a.href = clean; }); } }); }); }); observer.observe(document.body, { childList: true, subtree: true }); })(); } } catch(__e) { console.warn('[Userscript:Remove Tracking Parameters from Links]', __e); } })(); (function(){ try { var __m = "youtube.com"; var __re = new RegExp('^' + "youtube\\.com" + ', 'i'); if (__m === '*' || __re.test(location.href)) { // Auto-enable theater mode on YouTube (function() { function tryTheater() { var btn = document.querySelector('button[aria-label="Theater mode"], ytd-player #player button[title="Theater mode"]'); if (btn && !btn.classList.contains('activated')) { btn.click(); } } // Try immediately tryTheater(); // Try after navigation (SPA) var lastUrl = location.href; setInterval(function() { if (location.href !== lastUrl) { lastUrl = location.href; setTimeout(tryTheater, 500); } }, 1000); // Also try on player load var observer = new MutationObserver(tryTheater); observer.observe(document.body, { childList: true, subtree: true }); })(); } } catch(__e) { console.warn('[Userscript:YouTube Theater Mode Default]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + ', 'i'); if (__m === '*' || __re.test(location.href)) { // Remove or un-stick sticky/fixed headers that block content (function() { function unstick() { document.querySelectorAll('header, nav, [role="banner"], .header, .navbar, .sticky, .fixed-top, [style*="position: fixed"], [style*="position:sticky"]').forEach(function(el) { if (el.style.position === 'fixed' || el.style.position === 'sticky' || getComputedStyle(el).position === 'fixed' || getComputedStyle(el).position === 'sticky') { el.style.position = 'static'; el.style.top = 'auto'; el.style.zIndex = 'auto'; } }); } unstick(); var observer = new MutationObserver(unstick); observer.observe(document.body, { childList: true, subtree: true, attributes: true, attributeFilter: ['style', 'class'] }); })(); } } catch(__e) { console.warn('[Userscript:Kill Sticky Headers]', __e); } })(); })(); Add Refresh Token logic to auth manager docs by bugraoz93 · Pull Request #54196 · apache/airflow · GitHub
Skip to content

Add Refresh Token logic to auth manager docs - #54196

Merged
bugraoz93 merged 6 commits into
apache:mainfrom
bugraoz93:followup/51657/document-refresh-logic
Dec 15, 2025
Merged

Add Refresh Token logic to auth manager docs#54196
bugraoz93 merged 6 commits into
apache:mainfrom
bugraoz93:followup/51657/document-refresh-logic

Conversation

@bugraoz93

@bugraoz93bugraoz93 commented Aug 6, 2025

Copy link
Copy Markdown
Contributor

follow up docs
relates: #51657

^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in airflow-core/newsfragments.

vincbeck
vincbeck reviewed Aug 6, 2025
View reviewed changes
Comment threadairflow-core/docs/core-concepts/auth-manager/index.rst Outdated
@bugraoz93
bugraoz93force-pushed the followup/51657/document-refresh-logic branch from 5706a31 to 2ade6a3CompareAugust 7, 2025 23:52
vincbeck
vincbeck reviewed Aug 8, 2025
View reviewed changes
Comment threadairflow-core/docs/core-concepts/auth-manager/index.rst Outdated
Comment threadairflow-core/docs/core-concepts/auth-manager/index.rst Outdated
@bugraoz93
bugraoz93force-pushed the followup/51657/document-refresh-logic branch from 2ade6a3 to 9cf8339CompareAugust 8, 2025 17:12
pierrejeambrun
pierrejeambrun previously requested changes Aug 8, 2025
View reviewed changes

@pierrejeambrunpierrejeambrun left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We're currently discussing this. Just putting it on hold before we settle on the end goal.

cc: @potiuk@ashb

@github-actions

github-actionsBot commented Sep 23, 2025

Copy link
Copy Markdown
Contributor

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in 5 days if no further activity occurs. Thank you for your contributions.

@github-actionsgithub-actionsBot added the stale Stale PRs per the .github/workflows/stale.yml policy file label Sep 23, 2025
@bugraoz93

bugraoz93 commented Sep 23, 2025

Copy link
Copy Markdown
ContributorAuthor

Not stale, will update after the solution merged

@bugraoz93bugraoz93 removed the stale Stale PRs per the .github/workflows/stale.yml policy file label Sep 23, 2025
@github-actions

github-actionsBot commented Dec 3, 2025

Copy link
Copy Markdown
Contributor

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in 5 days if no further activity occurs. Thank you for your contributions.

@github-actionsgithub-actionsBot added the stale Stale PRs per the .github/workflows/stale.yml policy file label Dec 3, 2025
@bugraoz93bugraoz93 removed the stale Stale PRs per the .github/workflows/stale.yml policy file label Dec 3, 2025
@bugraoz93

bugraoz93 commented Dec 3, 2025

Copy link
Copy Markdown
ContributorAuthor

Okay it is already time. I will update the doc in a day or two. Finishing some CI work to lose dependency on v3 test branches on integration tests

@pierrejeambrun

pierrejeambrun commented Dec 4, 2025•
edited
Loading

Copy link
Copy Markdown
Member

I removed my request for change. Things have been discussed, implemented and stabilized, since then, we're ready for a doc update :)

pierrejeambrun
pierrejeambrun reviewed Dec 11, 2025
View reviewed changes
Comment threadairflow-core/docs/core-concepts/auth-manager/index.rst Outdated
@bugraoz93
bugraoz93force-pushed the followup/51657/document-refresh-logic branch 2 times, most recently from 6085e4f to 58b6ef5CompareDecember 11, 2025 19:22
bugraoz93
bugraoz93 commented Dec 11, 2025
View reviewed changes
Comment threadairflow-core/docs/core-concepts/auth-manager/index.rst Outdated
bugraoz93
bugraoz93 commented Dec 11, 2025
View reviewed changes
Comment threadairflow-core/docs/core-concepts/auth-manager/index.rst Outdated
vincbeck
vincbeck reviewed Dec 11, 2025
View reviewed changes
Comment threadairflow-core/docs/core-concepts/auth-manager/index.rst Outdated
Comment threadairflow-core/docs/core-concepts/auth-manager/index.rst
Comment threadairflow-core/docs/core-concepts/auth-manager/index.rst Outdated
Comment threadairflow-core/docs/core-concepts/auth-manager/index.rst Outdated
@bugraoz93
bugraoz93force-pushed the followup/51657/document-refresh-logic branch from 1e8f310 to adfd095CompareDecember 13, 2025 13:30
@bugraoz93bugraoz93 changed the title feat(doc): Add Refresh Token logic to auth manager docsAdd Refresh Token logic to auth manager docsDec 13, 2025
@bugraoz93
bugraoz93force-pushed the followup/51657/document-refresh-logic branch from adfd095 to d3ac537CompareDecember 13, 2025 13:33
@bugraoz93

bugraoz93 commented Dec 15, 2025

Copy link
Copy Markdown
ContributorAuthor

Docs and spellcheck are currently passing the error. Merging. Thanks for the reviews!

The image /mnt/ci-image-save-v3-linux_amd64-3.10.tar does not exist.

@bugraoz93
bugraoz93force-pushed the followup/51657/document-refresh-logic branch from d3ac537 to 912c5c9CompareDecember 15, 2025 21:03
@bugraoz93
bugraoz93 merged commit 1ba1732 into apache:mainDec 15, 2025
31 checks passed
@bugraoz93
bugraoz93 deleted the followup/51657/document-refresh-logic branch December 15, 2025 21:03
github-actionsBot pushed a commit that referenced this pull request Dec 15, 2025
@bugraoz93Your friendly bot
[v3-1-test] Add Refresh Token logic to auth manager docs (#54196)
1b3b825
* Update httponly logic in docs
(cherry picked from commit 1ba1732)
Co-authored-by: Bugra Ozturk <bugraoz93@users.noreply.github.com>
@github-actions

github-actionsBot commented Dec 15, 2025

Copy link
Copy Markdown
Contributor

Backport successfully created: v3-1-test

StatusBranchResult
✅v3-1-testPR Link

bugraoz93 added a commit that referenced this pull request Dec 15, 2025
@github-actions@bugraoz93
[v3-1-test] Add Refresh Token logic to auth manager docs (#54196) (#5…
cead7c6
…9482)
* Update httponly logic in docs
(cherry picked from commit 1ba1732)
Co-authored-by: Bugra Ozturk <bugraoz93@users.noreply.github.com>
TempestShaw pushed a commit to TempestShaw/airflow that referenced this pull request Dec 24, 2025
@bugraoz93@TempestShaw
Add Refresh Token logic to auth manager docs (apache#54196)
1bdac4f
* Update httponly logic in docs
ephraimbuddy pushed a commit that referenced this pull request Jan 6, 2026
@github-actions@bugraoz93@ephraimbuddy
[v3-1-test] Add Refresh Token logic to auth manager docs (#54196) (#5…
be75214
…9482)
* Update httponly logic in docs
(cherry picked from commit 1ba1732)
Co-authored-by: Bugra Ozturk <bugraoz93@users.noreply.github.com>
jhgoebbert pushed a commit to jhgoebbert/airflow_Owen-CH-Leung that referenced this pull request Feb 8, 2026
@bugraoz93@jhgoebbert
Add Refresh Token logic to auth manager docs (apache#54196)
698518d
* Update httponly logic in docs
Subham-KRLX pushed a commit to Subham-KRLX/airflow that referenced this pull request Mar 4, 2026
@bugraoz93@Subham-KRLX
Add Refresh Token logic to auth manager docs (apache#54196)
8282090
* Update httponly logic in docs
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@pierrejeambrunpierrejeambrunpierrejeambrun approved these changes
@jscheffljscheffljscheffl approved these changes
@vincbeckvincbeckvincbeck approved these changes
@potiukpotiukAwaiting requested review from potiuk

Assignees

No one assigned

Labels

kind:documentation

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@bugraoz93@pierrejeambrun@jscheffl@vincbeck