Skip to content

fix: redirect to detail page when creating a link that already exists - #224

Merged
mikeodr merged 1 commit into
mainfrom
mikeodr/fix-xsrf-160
Apr 7, 2026
Merged

fix: redirect to detail page when creating a link that already exists#224
mikeodr merged 1 commit into
mainfrom
mikeodr/fix-xsrf-160

Conversation

@mikeodr

@mikeodr mikeodr commented Mar 26, 2026

Copy link
Copy Markdown
Contributor

When attempting to create a link that already exists, redirect to the details page of the existing link to allow for updating.

Fixes #160

Comment thread golink.go Outdated
@mikeodr
mikeodr force-pushed the mikeodr/fix-xsrf-160 branch from 9b63181 to ec523ff Compare April 7, 2026 19:32
@mikeodr mikeodr changed the title golink: accept XSRF tokens from both home and detail page forms fix: redirect to detail page when creating a link that already exists Apr 7, 2026
@mikeodr
mikeodr force-pushed the mikeodr/fix-xsrf-160 branch from ec523ff to ee6663b Compare April 7, 2026 19:33
@mikeodr
mikeodr force-pushed the mikeodr/fix-xsrf-160 branch from ee6663b to 3b715d4 Compare April 7, 2026 19:49
When attempting to create a link that already exists, redirect
to the details page of the existing link to allow for updating.

Fixes #160

Signed-off-by: Mike O'Driscoll <mikeo@tailscale.com>
@mikeodr
mikeodr force-pushed the mikeodr/fix-xsrf-160 branch from 3b715d4 to b868a60 Compare April 7, 2026 19:50
@mikeodr

mikeodr commented Apr 7, 2026

Copy link
Copy Markdown
Contributor Author
image

@mikeodr
mikeodr merged commit 03a625e into main Apr 7, 2026
4 checks passed
@mikeodr
mikeodr deleted the mikeodr/fix-xsrf-160 branch April 7, 2026 19:52
@willnorris

Copy link
Copy Markdown
Member

that screenshot is a little weird because it says you can edit it below, but it's the non-editable version of the detail page :) Maybe first check if they have access to edit?

@mikeodr

mikeodr commented Apr 7, 2026

Copy link
Copy Markdown
Contributor Author

I was mixing users
image

Maybe yeah let me follow on.

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.

XSRF token mismatch error when using the "create" form to update a link

2 participants