Nieuwe RML Parser bug?

[RobIII]

Ik kan geen links plaatsen naar https://tio.run/##fY69DsI...Jv@QSziP7X/19g4F0ynC7p@xc

Zelfs als ik de url tussen "" zet werkt het niet:

test


Dit werkt

[ Voor 35% gewijzigd door RobIII op 12-09-2018 23:48 ]

[crisp]

Mogelijk keurt de parser de url af. Ik zal morgen even checken of dat zo is en waarom dan.

[RobIII]

crisp schreef op woensdag 12 september 2018 @ 23:49:
Mogelijk keurt de parser de url af.

Dat zou me niks verbazen met de dubbele ##

[crisp]

RobIII schreef op woensdag 12 september 2018 @ 23:50:
[...]

Dat zou me niks verbazen met de dubbele ##

Dat lijkt inderdaad de culprit te zijn. Ik zal de RFC er eens op naslaan

[crisp]

De tweede # escapen (%23) levert dit op:

https://tio.run/#%23fY69D...Jv@QSziP7X/19g4F0ynC7p@xc

[RobIII]

Thanks! Wa'n service zo, om dit tijdstip

[ Voor 85% gewijzigd door RobIII op 13-09-2018 00:17 ]

[crisp]

RobIII schreef op donderdag 13 september 2018 @ 00:17:
Thanks! Wa'n service zo, om dit tijdstip

yw

Overigens heb ik er geen probleem mee om in onze url-check \# te veranderen in \#+ voor deze use-case, maar dat is dan voor de volgende release

[.oisyn]

http://maarditwerktookniet.nl/#-#met-meerdere-pound-signs

\#+ is dus niet voldoende. Maar goed, de URLs zijn feitelijk invalid.

[ Voor 18% gewijzigd door .oisyn op 08-10-2018 15:51 ]

[ACM]

.oisyn schreef op maandag 8 oktober 2018 @ 15:45:
http://maarditwerktookniet.nl/#-#met-meerdere-pound-signs

\#+ is dus niet voldoende. Maar goed, de URLs zijn feitelijk invalid.

Inderdaad, want dit zijn de valide karakters volgens de RFC:

quote: https://tools.ietf.org/html/rfc3986#page-49

ALPHA DIGIT - . _ ~ ! $ & ' ( ) * + , ; = : @ / ?

Dus onder andere geen #, eigenlijk is deze verandering dus juist een bug-fix ipv een bug

[ Voor 7% gewijzigd door ACM op 08-10-2018 16:18 ]

[N8w8]

Voor de volledigheid, ik liep hier ook tegenaan, maar dan met # tekens die niet naast elkaar staan, bijv:
FreeVoipDeal
Bij vervangen met %23 maakt ie wel n link, maar met (uiteraard) ietwat verkeerde href.
(ik dacht mss helpt n voorbeeld van n invalid doch in de praktijk functionerende URL)

[ Voor 7% gewijzigd door N8w8 op 08-10-2018 17:47 . Reden: %23 verduidelijking ]

[.oisyn]

FreeVoipDeal

Vervangen met %23 werkt toch?
.edit: oh je bedoelt dat de site dan niet werkt. Tja... ze maken gewoon gebruik van een ongeldige syntax.

[ Voor 31% gewijzigd door .oisyn op 08-10-2018 17:10 ]

[crisp]

Het is inderdaad een ongeldige url. Ik zie op dit moment geen noodzaak om onze url-detectie uit te breiden om ook dergelijke malformed url's goed te keuren of proberen te fixen (want zoals het voorbeeld met FreeVoipDeal laat zien werkt dat ook niet altijd).

Tenzij dit een wijdverbreid gebruik wordt is dit wat mij betreft verder een wontfix.

[.oisyn]

Je kan er altijd nog een URL shortener overheen halen: https://bit.ly/2Oewt7S