• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

urish / firebase-server / 208 / 2
95%
master: 95%

Build:
DEFAULT BRANCH: master
Ran 14 May 2017 09:02PM UTC
Files 4
Run time 0s
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

14 May 2017 09:01PM UTC coverage: 97.0% (-0.3%) from 97.306%
208.2

push

travis-ci

urish
Properly fill in auth object for Android (#89)

* Properly fill in auth object for Android

The "auth" object is later used to do permission checks by the means of targaryen library [1].
I don't know the semantics of the token that JavaScript client sends, but [3] is for what Android sends in, and, given that JavaScript clients were tested to work correctly, I assume the format is different for some reason. The idea of this pull request is to bring the Android token in line with the official documentation at [2].

Why not make this change to targaryen library instead?
Well, the "auth" object supplied to targaryen is supposed to be of the format described in [2] rather than being protocol-bound, so firebase-server, as a protocol handler, is a more appropriate place to put an adapter in.

[1] https://github.com/goldibex/targaryen
[2] https://firebase.google.com/docs/reference/security/database/#auth
[3] https://github.com/urish/firebase-server/issues/88

* Use 'user_id', fallback to 'sub' in Android token

291 of 300 relevant lines covered (97.0%)

36.54 hits per line

Source Files on job 208.2
  • Tree
  • List 0
  • Changed 1
  • Source Changed 1
  • Coverage Changed 1
Coverage ∆ File Lines Relevant Covered Missed Hits/Line
  • Back to Build 208
  • Travis Job 208.2
  • 9a50b2c7 on github
  • Prev Job for on master (#203.2)
  • Next Job for on master (#209.3)
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2026 Coveralls, Inc