From 91d1d02616861b4ced4830bd4509d3bf3550a4ed Mon Sep 17 00:00:00 2001 From: AKP Date: Fri, 15 Apr 2022 21:21:52 +0100 Subject: Fix logic to determine if secure cookies should be generated When debug mode was enabled, it would fail to disable secure cookies. This fixes that and simplifies the logic used to determine if secure cookies should be enabled. --- CHANGELOG.md | 3 +++ walrss/internal/state/state.go | 7 +++---- 2 files changed, 6 insertions(+), 4 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 2a34834..1cb8db3 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -8,6 +8,9 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/), ### Added * Support for OPML imports and exports +### Fixed +* Secure cookies are no longer sent when debug mode is enabled + ## [0.1.0] - 2022-04-14 Initial release diff --git a/walrss/internal/state/state.go b/walrss/internal/state/state.go index af1d153..3ea8288 100644 --- a/walrss/internal/state/state.go +++ b/walrss/internal/state/state.go @@ -75,9 +75,8 @@ func (cfg *Config) GetHTTPAddress() string { } func (cfg *Config) EnableSecureCookies() bool { - enableSecureCookies := true - if !cfg.Debug { - enableSecureCookies = !cfg.Platform.DisableSecureCookies + if cfg.Debug { + return false } - return enableSecureCookies + return !cfg.Platform.DisableSecureCookies } -- cgit v1.2.3-70-g09d2