search

SHIV5T3R / CO-DE

An Open Source Collaborative Code Editor
36 stars 26 forks source link

Feature: cookie based auth #43

Closed ZakariaTalhami closed 4 months ago

ZakariaTalhami commented 10 months ago

Changes:

ZakariaTalhami commented 10 months ago

Is anyone aware of a better way to implement setting cookies in the header of the response?

believemanasseh commented 10 months ago

I'm not sure this change is needed again. The cookie-based authentication is being implemented with redis

zatoa commented 8 months ago

It could be a good idea to encrypt the cookie, a lot of auth libs do this, even if set to httponly

MrBisquit commented 8 months ago

It could be a good idea to encrypt the cookie, a lot of auth libs do this, even if set to httponly

If it's JWT there's no need for it