FlexMeasures / flexmeasures / 30077606381
87%

DEFAULT BRANCH: main
Ran
Jobs 1
Files 171
Run time 1min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 82.856% (+0.07%) from 82.787%
30077606381

push

github

web-flow 
Fix reading (boolean) settings from the environment, incl. SECURITY_TWO_FACTOR (#2341)

* fix: read SECURITY_TWO_FACTOR from the environment, as documented

The installation docs (and now the configuration docs, via PR #2340)
advise enabling 2FA with 'export SECURITY_TWO_FACTOR=True', but
read_env_vars never read that variable, so the app silently stayed
on the default (False). Parse it explicitly as a boolean, since env
values arrive as strings.

Signed-off-by: F.N. Claessen <claessen@seita.nl>

* docs: changelog entry for SECURITY_TWO_FACTOR env var fix

Signed-off-by: F.N. Claessen <claessen@seita.nl>

* fix: parse boolean settings from the environment properly

Environment values are strings, so any non-empty value - including
'False' or '0' - counted as enabled for DEBUG, and reached consumers
raw for MAIL_USE_TLS, MAIL_USE_SSL and FLEXMEASURES_JSON_COMPACT.
Coerce env values to booleans whenever the setting's default is a
boolean, and drop the now-redundant string special-casing for
FLEXMEASURES_JSON_COMPACT in app.py.

Signed-off-by: F.N. Claessen <claessen@seita.nl>

* fix: address review — docstring+strip for parse_bool_env, read SENTRY_DSN (keep SENTRY_SDN fallback), strict bool for json compactness; add tests

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Rjb8po3m8vyEvm7CnYVtwd

* fix: key boolean env coercion off DefaultConfig, broaden tests, clarify SENTRY_SDN is env-only

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Rjb8po3m8vyEvm7CnYVtwd

---------

Signed-off-by: F.N. Claessen <claessen@seita.nl>
Signed-off-by: Felix Claessen <30658763+Flix6x@users.noreply.github.com>
Co-authored-by: Claude Fable 5 <noreply@anthropic.com>

15847 of 19126 relevant lines covered (82.86%)

0.83 hits per line

Coverage Regressions

Lines Coverage ∆ File
36
77.78
0.8% flexmeasures/data/schemas/account.py
5
92.41
0.41% flexmeasures/ui/views/accounts.py
4
84.09
0.76% flexmeasures/api/common/schemas/users.py
2
98.17
8.45% flexmeasures/api/v3_0/accounts.py
Jobs
ID Job ID Ran Files Coverage
1 30077606381.1 171
82.86
GitHub Action Run
Source Files on build 30077606381