Corrected upstream stanza types.

googlearchive / go-gcm

GCM library for Go.

Apache License 2.0

103 stars 66 forks source link

Corrected upstream stanza types. #19

Closed bolshoy closed 8 years ago

bolshoy commented 8 years ago

apparently, Ack and Nack do not have a stanza type, fixed the relevant switch
added Ack to server receipts
exposed ack/nack/control/receipt constants to be used in the listener code

bolshoy commented 8 years ago

here is the ack message from the CCS, with no stanza type:

<message>
  <data:gcm xmlns:data="google:mobile:data">{"message_type":"ack","from":"cW_KBPUKj7k:APA91bEOj-_6uIMePkLi-dijWhM3OBbCOIGIFnXcVT-tN9DF2Dx3yQ9PDWCDKZbgrMnqtcF0wOXSnQS_iaydNU3UIMtO66AFWy-UFh24TI6qD820xSPNkdlpKd2zlYsGI0oknn8BjrFp","message_id":"6277050f-dd08-4c3e-92da-aabe558f0938"}
  </data:gcm>
</message>

silvolu commented 8 years ago

Thanks @bolshoy! Let me check if there is a reason for the stanza type difference, we might be able to fix it in the API. I'll probably end up merging this anyway given that is the correct thing to do now, but at least I'll be open an issue to track the upcoming change if needed.