Skip to content

Adds --https option#272

Merged
rochacbruno merged 1 commit intorochacbruno:mainfrom
cuducos:cuducos/https
May 27, 2025
Merged

Adds --https option#272
rochacbruno merged 1 commit intorochacbruno:mainfrom
cuducos:cuducos/https

Conversation

@cuducos
Copy link
Copy Markdown
Contributor

@cuducos cuducos commented May 26, 2025

As suggested by @osantana in #257, adding a --https config option.

Mostly for semantic reasons, I suggested keeping Marmite.https as Option<bool> instead of bool: this avoids a sort of incoherent state where url == "https://my.site" while https == false.

However I haven't implemented checks for other possible inconsistency (user can use --url https://my.site and --https false and in this case the https is ignored). This bit relies on the added docs (the help message for --https explicitly says it is used only if protocol is missing in the URL).

The changes in content.rs and site.rs are unrelated but suggested by cargo clippy — happy to revert, but I kept them since they are part of the contributing guide.

@rochacbruno rochacbruno merged commit 2c104ff into rochacbruno:main May 27, 2025
3 checks passed
@cuducos cuducos deleted the cuducos/https branch May 27, 2025 16:06
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