search

guyfedwards / nom

RSS reader for the terminal
GNU General Public License v3.0
372 stars 25 forks source link

specify minimum tls version #94

Closed Freyert closed 3 months ago

Freyert commented 4 months ago

I kept receiving 403s from https://fs.blog/feed/ . It turns out that it's probably because cloudflare blocks you unless you specify a minimum TLS version.

TLS 1.3 might be too high, but it solved my problem.

guyfedwards commented 4 months ago

Thanks for the contribution @Freyert. I think we should have the tls version be configurable as 1.2 is still supported. I suggest adding a minTLSVersion field to config.Feed which can then be used in Fetch.

Freyert commented 4 months ago

@guyfedwards agreed! I'll work it in this weekend.

guyfedwards commented 4 months ago

@Freyert are you happy with this? Looks good to me

guyfedwards commented 3 months ago

ping @Freyert

Freyert commented 3 months ago

👍 it's good for me.

guyfedwards commented 3 months ago

Thanks @Freyert