magne4000 / quassel-webserver

A web server/client for Quassel
MIT License
209 stars 38 forks source link

settings: Fix ignoring 'securecore: false' default #290

Closed digitalcircuit closed 3 years ago

digitalcircuit commented 3 years ago

In brief

Criteria Rank Reason
Impact ★☆☆ 1/3 Fixes non-default configuration of uncommon setting
Risk ★☆☆ 1/3 securecore value might not be set correctly
Intrusiveness ★☆☆ 1/3 Small set of changes, shouldn't interfere with other pull requests

Testing

Steps

  1. Set securecore: <value> in settings-user.json
    • Or, comment out the value like // securecore: value
  2. Restart/reload Quassel Webserver as needed to apply the new value
    • Verify with <URL to Quassel Webserver>/settings, e.g. http://localhost/chat/settings
  3. Open a fresh Private Browsing/Incognito tab
  4. Navigate to the Quassel Webserver instance
  5. Verify Local settings replaced by default settings (new version) is shown
  6. Expand more options
  7. Observe the checked state of the SSL core connection checkbox

Before

After

magne4000 commented 3 years ago

👍 Thank!