Skip to content

move client-side-redirect into web app part of kube-bind backend#436

Merged
cnvergence merged 5 commits into
kbind-dev:mainfrom
cnvergence:fix-client-side-redirect
Jan 23, 2026
Merged

move client-side-redirect into web app part of kube-bind backend#436
cnvergence merged 5 commits into
kbind-dev:mainfrom
cnvergence:fix-client-side-redirect

Conversation

@cnvergence

@cnvergence cnvergence commented Jan 2, 2026

Copy link
Copy Markdown
Member

Summary

Move redirect URL construction from backend to frontend.
Backend now stores redirect and session data and web app builds callback URL client-side.

What Type of PR Is This?

/kind feature

Related Issue(s)

Fixes #362

Release Notes

Moved client-side-redirect into web app part of kube-bind backend

@cnvergence
cnvergence force-pushed the fix-client-side-redirect branch from d835a54 to fcd1fb6 Compare January 9, 2026 13:34
Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>

On-behalf-of: @SAP karol.szwaj@sap.com
Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>

On-behalf-of: @SAP karol.szwaj@sap.com
@cnvergence cnvergence changed the title remove client-side-redirect in the backend move client-side-redirect from the backend Jan 20, 2026
@cnvergence
cnvergence marked this pull request as ready for review January 20, 2026 14:56
@cnvergence
cnvergence requested a review from a team as a code owner January 20, 2026 14:56
Comment thread web/src/views/Resources.vue Outdated
@xrstf

xrstf commented Jan 20, 2026

Copy link
Copy Markdown
Contributor

Just to be sure: "Moving from the backend" still means it's technically part of what kube-bind considers the backend, right? I thought you moved it into the kubectl plugin, but it's still in the backend, just the frontend part of the backend (oh us, naming is hard... :D)? I guess "client-side" would be just as confusing a term :D

@mjudeikis mjudeikis left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

nice one :)

Comment thread web/dist/assets/index.7407f299.js Outdated
Comment thread web/src/services/auth.ts Outdated
Comment thread web/src/services/auth.ts Outdated
Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>

On-behalf-of: @SAP karol.szwaj@sap.com
@cnvergence
cnvergence force-pushed the fix-client-side-redirect branch from fcd1fb6 to dfcecb3 Compare January 21, 2026 11:14
Comment thread web/dist/assets/index.6494c4af.js Outdated
Comment thread backend/client/client.go Outdated
coderabbitai[bot]
coderabbitai Bot previously approved these changes Jan 21, 2026
Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>

On-behalf-of: @SAP karol.szwaj@sap.com
Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>

On-behalf-of: @SAP karol.szwaj@sap.com
@cnvergence cnvergence changed the title move client-side-redirect from the backend move client-side-redirect into web app part of kube-bind backend Jan 21, 2026
@cnvergence
cnvergence merged commit 51185d9 into kbind-dev:main Jan 23, 2026
7 checks passed
@cnvergence
cnvergence deleted the fix-client-side-redirect branch January 23, 2026 12:28
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.

Fix client-side-redirect

3 participants