Remove leading slash from capi id

guardian / facia-scala-client

Low level client for the Facia JSON API

Other

2 stars 1 forks source link

Remove leading slash from capi id #204

Closed tomrf1 closed 6 years ago

tomrf1 commented 6 years ago

Users sometimes add capi backfill paths with a leading-slash, and facia-tool allows this. This breaks requests to capi preview because it creates urls with a double slash in the path, and this causes issues with api-gateway authorisation.

I'm fixing it here because it's used by both facia-press and mapi, and it's too late to fix it in facia-tool.

LATaylor-guardian commented 6 years ago

👍

mchv commented 6 years ago

Sounds sensible good 🐛 💀 @tomrf1 !