search

Aivean / royalroad-downloader

https://royalroad.com book downloader
MIT License
57 stars 4 forks source link

Improve URL Regex #21

Closed Pingger closed 3 years ago

Pingger commented 3 years ago

Hi,

just noticed that there is a slight inconvenience caused by the URL parameter. It allows: https://www.royalroad.com/fiction/xxxxx https://www.royalroad.com/fiction/xxxxx/title https://www.royalroad.com/fiction/xxxxx/title/

But does not allow: https://www.royalroad.com/fiction/xxxxx/ (trailing slash without title)

This PR aims to also allow this specific case. Unfortunately I can't run scala for some reason. And thus can't test if this actually works, BUT: The Parsing should not be impeded, as the newly allowed variant resolves to the same webpage.

Greetings

Aivean commented 3 years ago

Thank you for the contribution. I'll try to take a look and merge in the next few days.