] characters in the source link need to be escaped

toolforge / video2commons

Transfer video and audio from external sites to Commons.

https://video2commons.toolforge.org/

GNU General Public License v3.0

43 stars 24 forks source link

] characters in the source link need to be escaped #229

Open prototyperspective opened 2 weeks ago

prototyperspective commented 2 weeks ago

Minor issue: when there's a ] in the file title in the |source= field of the file info will be broken. It simply needs to be sanitized/escaped/replaced somehow. Example Example2 (there's many more since this is a widely used char).

royjor2016 commented 2 weeks ago

here to change

https://github.com/toolforge/video2commons/blob/c7a83336cc6720bdfe32dd705e51d63ce8fd756f/video2commons/frontend/urlextract.py#L219

prototyperspective commented 1 week ago

@royjor2016 Thanks for the info. If this is the place where to specify that could you change it? Don't know how to escape it and also not how to test it.