Skip to content

Conversation

kiingxo
Copy link

@kiingxo kiingxo commented Jul 17, 2025

Proposed Changes

  • Allow both http and https protocols for the server URL.
  • If the user does not specify a protocol, default to https://.
  • Prevents connection issues for users with self-hosted HTTP servers or strict HTTPS-only servers.

Related Issue(s)

Closes #860

Please check that the PR fulfills these requirements

  • Tests for the changes have been added (for bug fixes / features)
  • Set a 100 character limit in your editor/IDE to avoid white space diffs in the PR (run dart format .)
  • Updated/added relevant documentation (doc comments with ///).
  • Added relevant reviewers.

@rolandgeider
Copy link
Member

hey @kiingxo thanks for the PR!

We would need to use the normalizeServerUrl helper to lib/widgets/auth/server_field.dart since currently the validation would not work

Simulator Screenshot - iPhone 16 Plus - 2025-08-25 at 19 52 58

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.

An error occurred
2 participants