summaryrefslogtreecommitdiffhomepage
diff options
context:
space:
mode:
authorLinus Färnstrand <linus@mullvad.net>2018-10-12 12:41:47 +0200
committerLinus Färnstrand <linus@mullvad.net>2018-10-12 13:16:26 +0200
commit8523ccad75d07f8775084b62d7ae6ad5b7427bfe (patch)
tree71c75073ed1393b7ddb8acf6cbc990717f6a99d0
parent59886cc78785ae8486ac5689c8ed4440683aef17 (diff)
downloadmullvadvpn-8523ccad75d07f8775084b62d7ae6ad5b7427bfe.tar.xz
mullvadvpn-8523ccad75d07f8775084b62d7ae6ad5b7427bfe.zip
Make no mssfix also a valid mssfix
-rw-r--r--CHANGELOG.md3
-rw-r--r--gui/packages/desktop/src/renderer/components/AdvancedSettings.js2
2 files changed, 3 insertions, 2 deletions
diff --git a/CHANGELOG.md b/CHANGELOG.md
index d2d8451e3c..d4f9f346b0 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -30,8 +30,9 @@ Line wrap the file at 100 chars. Th
might have security issues.
### Fixed
-- Place Mssfix setting inside scrollable area
+- Place Mssfix setting inside scrollable area.
- Pick new random relay for each reconnect attempt instead of just retrying with the same one.
+- Fix so mssfix can be unset. Previously emptying the textbox did nothing.
#### Linux
- The app will have it's window resized correctly when display scaling settings are changed. This
diff --git a/gui/packages/desktop/src/renderer/components/AdvancedSettings.js b/gui/packages/desktop/src/renderer/components/AdvancedSettings.js
index e366776925..0c00886e68 100644
--- a/gui/packages/desktop/src/renderer/components/AdvancedSettings.js
+++ b/gui/packages/desktop/src/renderer/components/AdvancedSettings.js
@@ -181,7 +181,7 @@ export class AdvancedSettings extends Component<Props, State> {
_mssfixIsValid(): boolean {
const mssfix = this.state.editedMssfix;
- return mssfix >= MIN_MSSFIX_VALUE && mssfix <= MAX_MSSFIX_VALUE;
+ return mssfix === null || (mssfix >= MIN_MSSFIX_VALUE && mssfix <= MAX_MSSFIX_VALUE);
}
}