Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: remove server side cookie token methods #1742

Merged
merged 3 commits into from
Aug 28, 2024
Merged

Conversation

J0
Copy link
Contributor

@J0 J0 commented Aug 27, 2024

What kind of change does this PR introduce?

Remove set cookie tokens and clear cookie token methods as it looks like they aren't done server side.

Task: https://www.notion.so/supabase/team-auth-113cb19144c1419ea5fb1d600281d959?p=ff083dad758e44ef9b4e230804e0fee7&pm=s

@J0 J0 requested a review from a team as a code owner August 27, 2024 02:24
@J0 J0 changed the title feat: remove cookie token methods fix: remove server side cookie token methods Aug 27, 2024
@coveralls
Copy link

coveralls commented Aug 27, 2024

Pull Request Test Coverage Report for Build 10588318197

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 1 unchanged line in 1 file lost coverage.
  • Overall coverage decreased (-0.09%) to 57.885%

Files with Coverage Reduction New Missed Lines %
internal/api/auth.go 1 76.53%
Totals Coverage Status
Change from base Build 10553378997: -0.09%
Covered Lines: 9140
Relevant Lines: 15790

💛 - Coveralls

Copy link
Member

@kangmingtay kangmingtay left a comment

Choose a reason for hiding this comment

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

can we also remove the cookie config in configuration.go?

@J0
Copy link
Contributor Author

J0 commented Aug 28, 2024

Good catch, also removed the relevant test code in logout.go as it would otherwise be a no-op.

There's a use-cookie-header not sure if we need it atm / if it is linked so have left it there for now.

@J0 J0 merged commit c6efec4 into master Aug 28, 2024
2 checks passed
@J0 J0 deleted the j0/remove_set_cookie_tokens branch August 28, 2024 01:11
kangmingtay pushed a commit that referenced this pull request Aug 28, 2024
🤖 I have created a release *beep* *boop*
---


##
[2.159.2](v2.159.1...v2.159.2)
(2024-08-28)


### Bug Fixes

* allow anonymous user to update password
([#1739](#1739))
([2d51956](2d51956))
* hide hook name ([#1743](#1743))
([7e38f4c](7e38f4c))
* remove server side cookie token methods
([#1742](#1742))
([c6efec4](c6efec4))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
uxodb pushed a commit to uxodb/auth that referenced this pull request Nov 13, 2024
## What kind of change does this PR introduce?

Remove set cookie tokens and clear cookie token methods as it looks like
they aren't done server side.

Task:
https://www.notion.so/supabase/team-auth-113cb19144c1419ea5fb1d600281d959?p=ff083dad758e44ef9b4e230804e0fee7&pm=s
uxodb pushed a commit to uxodb/auth that referenced this pull request Nov 13, 2024
🤖 I have created a release *beep* *boop*
---


##
[2.159.2](supabase/auth@v2.159.1...v2.159.2)
(2024-08-28)


### Bug Fixes

* allow anonymous user to update password
([supabase#1739](supabase#1739))
([2d51956](supabase@2d51956))
* hide hook name ([supabase#1743](supabase#1743))
([7e38f4c](supabase@7e38f4c))
* remove server side cookie token methods
([supabase#1742](supabase#1742))
([c6efec4](supabase@c6efec4))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
LashaJini pushed a commit to LashaJini/auth that referenced this pull request Nov 13, 2024
## What kind of change does this PR introduce?

Remove set cookie tokens and clear cookie token methods as it looks like
they aren't done server side.

Task:
https://www.notion.so/supabase/team-auth-113cb19144c1419ea5fb1d600281d959?p=ff083dad758e44ef9b4e230804e0fee7&pm=s
LashaJini pushed a commit to LashaJini/auth that referenced this pull request Nov 13, 2024
🤖 I have created a release *beep* *boop*
---


##
[2.159.2](supabase/auth@v2.159.1...v2.159.2)
(2024-08-28)


### Bug Fixes

* allow anonymous user to update password
([supabase#1739](supabase#1739))
([2d51956](supabase@2d51956))
* hide hook name ([supabase#1743](supabase#1743))
([7e38f4c](supabase@7e38f4c))
* remove server side cookie token methods
([supabase#1742](supabase#1742))
([c6efec4](supabase@c6efec4))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
LashaJini pushed a commit to LashaJini/auth that referenced this pull request Nov 15, 2024
## What kind of change does this PR introduce?

Remove set cookie tokens and clear cookie token methods as it looks like
they aren't done server side.

Task:
https://www.notion.so/supabase/team-auth-113cb19144c1419ea5fb1d600281d959?p=ff083dad758e44ef9b4e230804e0fee7&pm=s
LashaJini pushed a commit to LashaJini/auth that referenced this pull request Nov 15, 2024
🤖 I have created a release *beep* *boop*
---


##
[2.159.2](supabase/auth@v2.159.1...v2.159.2)
(2024-08-28)


### Bug Fixes

* allow anonymous user to update password
([supabase#1739](supabase#1739))
([2d51956](supabase@2d51956))
* hide hook name ([supabase#1743](supabase#1743))
([7e38f4c](supabase@7e38f4c))
* remove server side cookie token methods
([supabase#1742](supabase#1742))
([c6efec4](supabase@c6efec4))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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.

3 participants