@ in homepage toegestaan

[Spider.007]

Officieel mag het natuurlijk, maar het klopt toch niet helemaal; misschien is daar 'iets' aan te doen? Vooral omdat Firefox nu ook begint te zeuren zodra er erop klikt

[Verwijderde gebruiker]

[crisp]

Het is dezelfde check die React doet voor bijvoorbeeld iconurl, daar wordt ook enkel gekeken of het begint met http:// en of er geen spaties in staan

[Spider.007]

Uhu, dat weet ik

[moto-moi]

Hmm, je zou natuurlijk de check zo kunnen maken dat hij wel http://user:password@blaat toestaat maar niet http://user@blaat , de eerste vind ik namelijk nog wel z'n nut hebben.

[Spider.007]

Ja, maar ik denk dat het nuttige gebruik daarvan beperkt zal zijn; ik kan me niet voorstellen dat men zijn homepage afschermt met username en wachtwoord. Ik zou het er gewoon helemaal uitgooien en vervangen door de regexp die al voor de RML parser wordt gebruikt; maar dan zonder het authenticatie-gedeelte.

[moto-moi]

De vraag waarom er meerdere regexp's in react zitten voor dezelfde content is zeker flauw he?

[Spider.007]

Ik gok dat toen de homepage als custom-userfield werd afgescheiden dat de regexp toen versimpelt is om als voorbeeld te kunnen dienen.

[Erkens]

Spider.007 schreef op donderdag 21 februari 2008 @ 09:41:
Officieel mag het natuurlijk, maar het klopt toch niet helemaal; misschien is daar 'iets' aan te doen? Vooral omdat Firefox nu ook begint te zeuren zodra er erop klikt

[Verwijderde gebruiker]

Maar er is imo niks mis mee? En het is juist mooi dat je een waarschuwing van je browser krijgt Ik zou niet weten waarom het geblokeerd moet worden (zolang het uiteraard gaat om een linkje die geplaatst wordt).

[Spider.007]

In dit specifieke geval gaat het om een gebruiker die zijn emailadres ook in het homepage-field heeft geplakt; iets wat imo voorkomen moet worden indien mogelijk

[Erkens]

Spider.007 schreef op donderdag 21 februari 2008 @ 12:31:
In dit specifieke geval gaat het om een gebruiker die zijn emailadres ook in het homepage-field heeft geplakt; iets wat imo voorkomen moet worden indien mogelijk

wat je kan doen door verplicht te stellen dat die http:// ook in het veld moet staan ipv dat dat erbij geplakt wordt

[crisp]

Feit blijft dat het op zich (met http:// er voor geplakt) een geldig URL is. Ik heb nu een nette check_url functie gemaakt en een extra check ingebouwd op user-credentials in de URL indien valid. Voor homepage staan we dat niet (meer) toe.