Skip to content

Fix server.cookieSecret config option.#1603

Merged
tvst merged 1 commit intostreamlit:developfrom
tvst:fix-cookie-config
Jun 18, 2020
Merged

Fix server.cookieSecret config option.#1603
tvst merged 1 commit intostreamlit:developfrom
tvst:fix-cookie-config

Conversation

@tvst
Copy link
Copy Markdown
Contributor

@tvst tvst commented Jun 18, 2020

The config option...

  1. ...shouldn't read environment variables. This happens automatically.
  2. ...should specify a type.
  3. ...doesn't have to include a test for its specific environment variable. Environment variables and the config system are already tested elsewhere.

1. It shouldn't read environment variables. This happens automatically.
2. It should have a type.
3. Environment variables and the config system are already tested, so no
   need to make sure it also works in the case of STREAMLIT_SERVER_COOKIE_SECRET.
@tvst tvst requested review from Amey-D and karriebear June 18, 2020 06:21
@tvst tvst requested a review from a team as a code owner June 18, 2020 06:21
@tvst tvst merged commit 96f6b18 into streamlit:develop Jun 18, 2020
@tvst tvst deleted the fix-cookie-config branch June 18, 2020 18:52
tconkling added a commit that referenced this pull request Jun 22, 2020
* develop:
  Formatting changes from `make pyformat` (#1621)
  Add config to disable CSRF + conflict check with CORS (#1571)
  MediaFile Grace Period using KEEP_DELAY = 5 seconds (#1494)
  Fix server.cookieSecret config option. (#1603)
  Create a config option to toggle websocket compression (#1544)
  Remove Python 2 and Python 3.5 staleness (#1600)
  Disable/Remove scroll when in full screen mode (#1586)
  Better welcome message when running `streamlit hello` (#1585)
  Scale image to approximate size on screen (#1594)
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.

2 participants