Skip to content

GitLab

  • Projects
  • Groups
  • Snippets
  • Help
    • Loading...
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
vidjil
vidjil
  • Project overview
    • Project overview
    • Details
    • Activity
    • Releases
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 1,692
    • Issues 1,692
    • List
    • Boards
    • Labels
    • Service Desk
    • Milestones
  • Merge Requests 85
    • Merge Requests 85
  • CI / CD
    • CI / CD
    • Pipelines
    • Jobs
    • Schedules
  • Operations
    • Operations
    • Incidents
    • Environments
  • Packages & Registries
    • Packages & Registries
    • Container Registry
  • Analytics
    • Analytics
    • CI / CD
    • Repository
    • Value Stream
  • Snippets
    • Snippets
  • Members
    • Members
  • Collapse sidebar
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
  • vidjil
  • vidjilvidjil
  • Issues
  • #2192

Closed
Open
Opened Feb 18, 2017 by Mathieu Giraud@magiraudOwner

API/URL: être robuste à l'interprétation d'URL non correctement formées

Dans model_loader.js, pour l'analyse de l'URL, on a pas mal de choses qui peuvent être cassées si l'URL n'est pas bien formée, que ce soit des choses anciennes (clones=tmp2[1].split(',')) ou des choses mises récemment par @aurelBZH (sp.splitY=tmp2[1].split(',')[1]).

L'URL venant "de l'extérieur", on devrait être robuste à des URL pas très bien formées, et charger le mieux possible et/ou mettre un warning, en tout cas ne pas faire tout planter.

(Quelque part, si quelqu'un donneplot=axe, ne devrait-on pas l'interpréter comme plot=axe,axeypardefaut,grid ? Voir aussi #2191 (closed).)

cc @aurelBZH @RyanHerb @mikael-s

To upload designs, you'll need to enable LFS and have admin enable hashed storage. More information
Assignee
Assign to
None
Milestone
None
Assign milestone
Time tracking
None
Due date
None
Reference: vidjil/vidjil#2192