coolaj86
/
telebit.js
1
2
Fork 1
Code Issues 46 Pull-Requests 2 Releases Aktivität

better param parsing #36

Zusammengeführt
coolaj86 hat 1 Commits von params-parsing nach master 2019-05-29 21:38:17 +00:00 zusammengeführt
Diskussion 1 Commits 1 Geänderte Dateien 1 +1 -1
coolaj86 hat 2019-05-28 23:09:55 +00:00 kommentiert
Besitzer
Link kopieren

Removing any and all of the following leading characters before parsing the params: #/?.

  • #access_token=x&foo=bar => access_token=x&foo=bar
  • #/?access_token=x&foo=bar => access_token=x&foo=bar
  • #/access_token=x&foo=bar => access_token=x&foo=bar
  • #?access_token=x&foo=bar => access_token=x&foo=bar
Removing any and all of the following leading characters before parsing the params: `#/?`. * `#access_token=x&foo=bar` => `access_token=x&foo=bar` * `#/?access_token=x&foo=bar` => `access_token=x&foo=bar` * `#/access_token=x&foo=bar` => `access_token=x&foo=bar` * `#?access_token=x&foo=bar` => `access_token=x&foo=bar`
Ghost hat die Änderungen 2019-05-29 01:49:48 +00:00 genehmigt
lib/html/js/app.js
@ -4,3 +4,3 @@
document.body.hidden = false;
var hash = window.location.hash.substr(1);
var hash = window.location.hash.replace(/^[\/#?]+/, '');
Ghost hat 2019-05-29 01:49:42 +00:00 kommentiert
Erstmaliger Mitwirkender
Link kopieren

Would #? to be safe, but I think inside a square bracket it's fine.

Would \#\? to be safe, but I think inside a square bracket it's fine.
coolaj86 hat 2019-05-29 21:38:30 +00:00 kommentiert
Autor
Besitzer
Link kopieren

true on both accounts

true on both accounts
coolaj86 hat diesen Pull-Request 2019-05-29 21:38:17 +00:00 geschlossen
coolaj86 hat 2019-05-31 22:19:49 +00:00 kommentiert
Autor
Besitzer
Link kopieren

Thanks @mikegwhit

Thanks @mikegwhit
Anmelden, um an der Diskussion teilzunehmen.
Reviewer
Keine Reviewer
Label
Label entfernen   
feature

something new that would be great to add

Kein Label
feature
Meilenstein
Meilenstein entfernen
Keine Einträge
Kein Meilenstein
Zuständig
Zuständige entfernen
Niemand zuständig
2 Beteiligte
Nachrichten
Fällig am
Das Fälligkeitsdatum ist ungültig oder außerhalb des zulässigen Bereichs. Bitte verwende das Format „jjjj-mm-tt“.

Kein Fälligkeitsdatum gesetzt.

Abhängigkeiten

Keine Abhängigkeiten gesetzt.

Referenz: coolaj86/telebit.js#36
Keine Beschreibung angegeben.
Branch "params-parsing" löschen

Das Löschen eines Branches ist permanent. Obwohl der Branch für eine kurze Zeit weiter existieren könnte, kann diese Aktion in den meisten Fällen NICHT rückgängig gemacht werden. Fortfahren?