Polymer: rootPath estará incorreto se o URL incluir queryParameters ou fragmento com barra

Criado em 1 set. 2017  ·  3Comentários  ·  Fonte: Polymer/polymer

Essa linha precisa remover os parâmetros de consulta e o identificador de fragmento antes de procurar a barra final. Se o local da minha página for " http: // localhost : 8081 / files? Foo = abc / def", o código existente define rootPath como " http: // localhost : 8081 / files? Foo = abc /", mas deveria ser " http: // localhost : 8081 /".

https://github.com/Polymer/polymer/blob/1a32f4dd423116c5d1ee13e452922839d4eebd60/lib/utils/resolve-url.html#L90

bug p3 wontfix

Comentários muito úteis

Um pouco de programação defensiva para se comportar melhor nesta situação seria bom. Ignorar dados inválidos ou gerar um erro explícito seria melhor do que transmitir silenciosamente resultados inválidos até que eles causem um problema muito mais tarde que precise ser rastreado.

Todos 3 comentários

IMHO, o URL http://localhost:8181/files?foo=abc/def não tem uma sintaxe válida . O parâmetro de consulta foo=abc/def contém um delimitador de caminho. Se você precisar passar uma lista, o delimitador é ponto-e-vírgula ( ; ), caso contrário, se você tiver acesso ao backend que implementa a API REST, pode substituir a barra por um traço ( - ) .

Um pouco de programação defensiva para se comportar melhor nesta situação seria bom. Ignorar dados inválidos ou gerar um erro explícito seria melhor do que transmitir silenciosamente resultados inválidos até que eles causem um problema muito mais tarde que precise ser rastreado.

Este problema foi marcado automaticamente como obsoleto porque não teve atividades recentes. Ele será fechado se nenhuma outra atividade ocorrer. Obrigado por suas contribuições.

Esta página foi útil?
0 / 5 - 0 avaliações

Questões relacionadas

yordis picture yordis  ·  3Comentários

myuseringithub picture myuseringithub  ·  3Comentários

paranoid-android picture paranoid-android  ·  3Comentários

hzmnet picture hzmnet  ·  4Comentários

limonte picture limonte  ·  3Comentários