Title: Incorrect timestamp format in AndroidNotification.event_time · Issue #337 · firebase/firebase-admin-java · GitHub
Open Graph Title: Incorrect timestamp format in AndroidNotification.event_time · Issue #337 · firebase/firebase-admin-java
X Title: Incorrect timestamp format in AndroidNotification.event_time · Issue #337 · firebase/firebase-admin-java
Description: I think the way we format the event_time field on AndroidNotification is wrong: firebase-admin-java/src/main/java/com/google/firebase/messaging/AndroidNotification.java Lines 458 to 462 in 83b7a64 public Builder setEventTimeInMillis(long...
Open Graph Description: I think the way we format the event_time field on AndroidNotification is wrong: firebase-admin-java/src/main/java/com/google/firebase/messaging/AndroidNotification.java Lines 458 to 462 in 83b7a64 ...
X Description: I think the way we format the event_time field on AndroidNotification is wrong: firebase-admin-java/src/main/java/com/google/firebase/messaging/AndroidNotification.java Lines 458 to 462 in 83b7a64 ...
Opengraph URL: https://github.com/firebase/firebase-admin-java/issues/337
X: @github
Domain: github.com
{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"Incorrect timestamp format in AndroidNotification.event_time","articleBody":"I think the way we format the `event_time` field on `AndroidNotification` is wrong:\r\n\r\nhttps://github.com/firebase/firebase-admin-java/blob/83b7a64c103fbf20f46c04df229000529e9a8d2d/src/main/java/com/google/firebase/messaging/AndroidNotification.java#L458-L462\r\n\r\nThere seems to be couple of issues with this.\r\n\r\n1. We are not specifying the timezone for the `SimpleDateFormat`. Therefore the resulting timestamp string is in the local timezone while the backend expects it to be in UTC.\r\n2. The way nanoseconds are formatted seems to be wrong. For the milliseconds value `1546304523123L` this generates the string `2019-01-01T01:02:03.000000123Z`. I would expect the nanos to be formatted as `123000000` (i.e. 123 milliseconds in nanos).","author":{"url":"https://github.com/hiranya911","@type":"Person","name":"hiranya911"},"datePublished":"2019-12-18T21:52:54.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":1},"url":"https://github.com/337/firebase-admin-java/issues/337"}
| route-pattern | /_view_fragments/issues/show/:user_id/:repository/:id/issue_layout(.:format) |
| route-controller | voltron_issues_fragments |
| route-action | issue_layout |
| fetch-nonce | v2:960c4558-dbdb-d463-2a80-5a35ccafe327 |
| current-catalog-service-hash | 81bb79d38c15960b92d99bca9288a9108c7a47b18f2423d0f6438c5b7bcd2114 |
| request-id | 9864:FF800:D47C7B:1142F85:698E9235 |
| html-safe-nonce | f1299b0467c439e78052d26aa9a1b527f9d618de21cefc26c6abc5842be9a817 |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiI5ODY0OkZGODAwOkQ0N0M3QjoxMTQyRjg1OjY5OEU5MjM1IiwidmlzaXRvcl9pZCI6Ijg1MDE2Nzc5MTIxMDA4Njg2NjEiLCJyZWdpb25fZWRnZSI6ImlhZCIsInJlZ2lvbl9yZW5kZXIiOiJpYWQifQ== |
| visitor-hmac | b78dfdd0a41379fac25e23dc1095a8a467b3b7765d69643f1b570556d3019f94 |
| hovercard-subject-tag | issue:539944706 |
| github-keyboard-shortcuts | repository,issues,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/_view_fragments/issues/show/firebase/firebase-admin-java/337/issue_layout |
| twitter:image | https://opengraph.githubassets.com/3517eee0aa14f6aa7e765f38dd676129812b82d7932df3998193fa6e1995f71f/firebase/firebase-admin-java/issues/337 |
| twitter:card | summary_large_image |
| og:image | https://opengraph.githubassets.com/3517eee0aa14f6aa7e765f38dd676129812b82d7932df3998193fa6e1995f71f/firebase/firebase-admin-java/issues/337 |
| og:image:alt | I think the way we format the event_time field on AndroidNotification is wrong: firebase-admin-java/src/main/java/com/google/firebase/messaging/AndroidNotification.java Lines 458 to 462 in 83b7a64 ... |
| og:image:width | 1200 |
| og:image:height | 600 |
| og:site_name | GitHub |
| og:type | object |
| og:author:username | hiranya911 |
| hostname | github.com |
| expected-hostname | github.com |
| None | cb2828a801ee6b7be618f3ac76fbf55def35bbc30f053a9c41bf90210b8b72ba |
| turbo-cache-control | no-preview |
| go-import | github.com/firebase/firebase-admin-java git https://github.com/firebase/firebase-admin-java.git |
| octolytics-dimension-user_id | 1335026 |
| octolytics-dimension-user_login | firebase |
| octolytics-dimension-repository_id | 84979119 |
| octolytics-dimension-repository_nwo | firebase/firebase-admin-java |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 84979119 |
| octolytics-dimension-repository_network_root_nwo | firebase/firebase-admin-java |
| turbo-body-classes | logged-out env-production page-responsive |
| disable-turbo | false |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | e6b91a7e6e46287d26887e3fb7a4161657bab8f7 |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width