search

mozilla-sensorweb / sensorweb-server

Mozilla Public License 2.0
3 stars 6 forks source link

[Issue #67] Embed user auth query params inside auth token payload #69

Closed ferjm closed 7 years ago

ferjm commented 7 years ago

As discussed on IRC, this PR embeds all query params inside the token payload, so all of them are properly signed.

I also updated the doc. Although it will change significantly in following PRs (check #68 for instance).

coveralls commented 7 years ago

Coverage Status

Coverage increased (+0.04%) to 85.638% when pulling 6797a2286ae75de2949b885ff85039844fc49bef on ferjm:authheader into 47530b78855adb6e39f53f8c8f6ae4fd1e60fffb on mozilla-sensorweb:master.

coveralls commented 7 years ago

Coverage Status

Coverage increased (+0.04%) to 85.638% when pulling 6797a2286ae75de2949b885ff85039844fc49bef on ferjm:authheader into 47530b78855adb6e39f53f8c8f6ae4fd1e60fffb on mozilla-sensorweb:master.

ferjm commented 7 years ago

r? @julienw

coveralls commented 7 years ago

Coverage Status

Coverage increased (+0.1%) to 85.714% when pulling f89d0d62e9b4812e556a19b2aeca5e43f9509eb6 on ferjm:authheader into 47530b78855adb6e39f53f8c8f6ae4fd1e60fffb on mozilla-sensorweb:master.

ferjm commented 7 years ago

r? @julienw I just updated the PR

coveralls commented 7 years ago

Coverage Status

Coverage increased (+0.08%) to 85.676% when pulling add291819e792bb3fedb6e010167cabf7920c396 on ferjm:authheader into 47530b78855adb6e39f53f8c8f6ae4fd1e60fffb on mozilla-sensorweb:master.

ferjm commented 7 years ago

r? @julienw