Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Switch from http to https #482

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from
Open

Switch from http to https #482

wants to merge 1 commit into from

Conversation

pabs3
Copy link
Contributor

@pabs3 pabs3 commented Feb 15, 2023

Switch everything from http to https. Also fix a typo.

@soimort
Copy link
Owner

soimort commented Feb 15, 2023

Have you tested if it actually works?
gawk does not have built-in support for TLS. AFAIK simply changing port 80 to 443 in the TCP header will not work.

@soimort
Copy link
Owner

soimort commented Feb 15, 2023

The https thing aside, the spelling fix looks very reasonable to me. Could you open a separate PR (only for the 2nd commit)?

@pabs3
Copy link
Contributor Author

pabs3 commented Feb 16, 2023 via email

@pabs3 pabs3 changed the title Switch from http to https, fix typo Switch from http to https Feb 16, 2023
@soimort
Copy link
Owner

soimort commented Feb 23, 2023

curl is used whenever gawk cannot handle a request (such as the HTTPS ones), but as it is an external process there is a performance overhead.

I think the best way to implement this is to have an option --secure to force the usage of curl (hence HTTPS) for all requests; defaults to plain HTTP/1.1 otherwise.

If Google Translate starts to require HTTPS one day in the future and gawk can't handle it, maybe it's then time to rewrite trans in a more modern language with a decent standard library. (Requiring HTTP/2 or HTTP/3 is a bit too radical at this point, though)

@pabs3
Copy link
Contributor Author

pabs3 commented Feb 24, 2023 via email

Prevents attackers from modifying responses and reading translation data.

Suggested-by: opensnitch & check-all-the-things
@pabs3
Copy link
Contributor Author

pabs3 commented Mar 12, 2023 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants