Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Invalid VAPID token: A value in the vapid claims is either missing or incorrectly specified #195

Open
atkawa7 opened this issue Jul 2, 2022 · 15 comments

Comments

@atkawa7
Copy link

atkawa7 commented Jul 2, 2022

When using this library I am getting Invalid VAPID token. This occurs randomly sometimes you won't get it at all. Sometimes you do.

 [main] DEBUG *.*.*.PushService - header Authorization WebPush <jwt>
16:43:57.748 [main] DEBUG *.*.*.PushService - header Content-Encoding aesgcm
16:43:57.748 [main] DEBUG *.*.*.PushService - header Encryption salt=<salt>
16:43:57.748 [main] DEBUG *.*.*.PushService - header TTL 2419200
16:43:57.748 [main] DEBUG *.*.*.PushService - header Crypto-Key dh=BHGQ****dw=;p256ecdsa=BB_****
16:43:57.748 [main] DEBUG *.*.*.PushService - header Content-Type application/octet-stream

The response I am getting is

{"origin":"remote","type":"response","correlation":"b7b0107273db5d1e","duration":2227,"protocol":"HTTP/1.1","status":401,"headers":{"Connection":["keep-alive"],"Content-Length":["296"],"Content-Type":["application/json"],"Date":["Sat, 02 Jul 2022 14:44:00 GMT"],"Server":["nginx"]},"body":{"code":401,"errno":109,"error":"Unauthorized","message":"Invalid VAPID token: A value in the vapid claims is either missing or incorrectly specified (e.g. \"exp\":\"12345\" or \"sub\":null). Please correct and retry.","more_info":"http://autopush.readthedocs.io/en/latest/http.html#error-codes"}}
Subscription subscription = new Subscription();
subscription.endpoint = endpoint;
subscription.keys = new Keys();
subscription.keys.auth = auth;
subscription.keys.p256dh = p256dh;

ObjectMapper mapper = new ObjectMapper();
var webPushMessage = new WebPushMessage();
webPushMessage.setBody("A new document was uploaded");
webPushMessage.setTitle("Document uploaded");
var notification = new Notification(subscription, mapper.writeValueAsString(webPushMessage));

var logbook = createLogbook();
var client = httpClient(logbook);
var pushService = new PushService(keys.getPublicKey(), keys.getPrivateKey(), client);
pushService.send(notification);
@atkawa7
Copy link
Author

atkawa7 commented Jul 2, 2022

Ended up copying the lib and changing the jose library to the common nimbus and those issues are now gone.

@aistomin
Copy link

@atkawa7 I have the same problem as you, but I'm not sure I understood your solution. "changing the jose library to the common nimbus" - did you change it in the source code and re-built the jar?

@atkawa7
Copy link
Author

atkawa7 commented Jul 31, 2022

@aistomin Yes that's what I did. I think you only need to changed three or 4 lines and exception. Something along these lines

            var builder = new Builder();
            builder.audience(notification.getOrigin());
            builder.expirationTime(Date.from(Instant.now().plus(12, ChronoUnit.HOURS)));
            if (getSubject() != null) {
                builder.subject(getSubject());
            }


            var payload = new Payload(builder.build().toJSONObject());
            var jwsObject = new JWSObject(new JWSHeader(ES256), payload);
            if (privateKey instanceof ECPrivateKey ecPrivateKey) {
                var jwsSigner = new ECDSASigner(ecPrivateKey);
                jwsObject.sign(jwsSigner);
            } else {
                throw new JOSEException("Expected ECPrivateKey");
            }

@aistomin
Copy link

aistomin commented Aug 3, 2022

@atkawa7 sorry, one more question, did you use https://mvnrepository.com/artifact/com.nimbusds/nimbus-jose-jwt/9.23 ?

@atkawa7
Copy link
Author

atkawa7 commented Aug 3, 2022

@aistomin Yes that's correct

@chamallamudi-gopikrishna

@atkawa7 Sorry, Please Provide code line where to use? I have same problem. Especially imports

@umutawakil
Copy link

@atkawa7 Hi. Could you expand on your answer? Does this mean you removed the jwt code and replaced it with a 3rd party to generate the token? If so, do you know why the current code is not building a token Mozilla likes? I dont get the error you were getting with chrome but I do trying to push to the Firefox/Mozilla push service. I wonder what made this stop working

@atkawa7
Copy link
Author

atkawa7 commented Sep 23, 2022

@umutawakil Yes that is correct. I removed jose4j and replaced it with nimbus-jose-jwt. @chamallamudi-gopikrishna

import com.nimbusds.jose.JOSEException;
import com.nimbusds.jose.JWSHeader;
import com.nimbusds.jose.JWSObject;
import com.nimbusds.jose.Payload;
import com.nimbusds.jose.crypto.ECDSASigner;
import com.nimbusds.jwt.JWTClaimsSet.Builder;
import static com.nimbusds.jose.JWSAlgorithm.ES256;

@sbodmer
Copy link

sbodmer commented Oct 21, 2022

I have the same problem, I tried with the solution explained using nimbus, here is the code
that I changed in AbstractPushService (and I adapted the rest for the JOSEException)...
It still works with chrome, but not with firefox. Is there something else to modify ?

JWTClaimsSet.Builder builder = new Builder();
builder.audience(notification.getOrigin());
builder.expirationTime(Date.from(Instant.now().plus(12, ChronoUnit.HOURS)));
builder.
if (getSubject() != null) {
    builder.subject(getSubject());
}

Payload payload = new Payload(builder.build().toJSONObject());
JWSObject jws = new JWSObject(new JWSHeader(ES256), payload);
if (privateKey instanceof ECPrivateKey) {
  ECPrivateKey ecPrivateKey = (ECPrivateKey) privateKey;
  ECDSASigner jwsSigner = new ECDSASigner(ecPrivateKey);
  jws.sign(jwsSigner);

} else {
  throw new JOSEException("Expected ECPrivateKey");
}

byte[] pk = Utils.encode((ECPublicKey) getPublicKey());
if (encoding == Encoding.AES128GCM) {
  headers.put("Authorization", "vapid t=" + jws.serialize() + ", k=" + Base64.getUrlEncoder().withoutPadding().encodeToString(pk));
} else if (encoding == Encoding.AESGCM) {
  headers.put("Authorization", "WebPush " + jws.serialize());
}

The error in firefox is still a 401 (Invalid VAPID tokens: A value in the vapid claims is either missing or incorrectly specified)

@aistomin
Copy link

@sbodmer for me it turned out that the problem was not in the library at all. From experience I found out that Firefox requires sub parameter in the notification. When I started providing email as sub parameter "Invalid VAPID token" problem never happened to me with the latest version of the webpush-java.

@sbodmer
Copy link

sbodmer commented Oct 21, 2022

Bingo, I added the "sub" part and now it works, thanks a lot my friend ;^)
I read the spec and the "sub" is not mandatory...

@aistomin
Copy link

@sbodmer I'm glad it helped.

I read the spec and the "sub" is not mandatory...

Yes, absolutely. Looks like it's either a Firefox bug, or intentional violation of the protocol :)

@guss77
Copy link

guss77 commented Sep 5, 2023

@aistomin, @sbodmer - can you please show what you changed in the code? I'm at a loss as to where to add this "sub" parameter.

@sbodmer
Copy link

sbodmer commented Sep 6, 2023

You just have to add the Subject in your push service, nothing else changes...

Here is what I changed and solved the problem (but it was a long time ago, I'm not entirely sure...)
push = new PushService();
push.setPrivateKey(vapiPrivateKey);
push.setPublicKey(vapiPublicKey);
push.setSubject("mailto:xxx"); //--- Needed for firefox

What was odd is that in the specification the "Subject" is not mandatory, but Firefox (not chrome) makes it mandatory, so they are violating the specifications.

Good luck

@NickM-27
Copy link

Just to add to this, I found another Firefox issue. The exp must be an int. Both Google and Apple handle float just fine but for Firefox it can not have a decimal or it returns the same 401

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

7 participants