Skip to content

SHAKEN VS request fails for "URI parsing failed" #168

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

Open
di-shi opened this issue Jan 24, 2023 · 12 comments
Open

SHAKEN VS request fails for "URI parsing failed" #168

di-shi opened this issue Jan 24, 2023 · 12 comments

Comments

@di-shi
Copy link

di-shi commented Jan 24, 2023

We run SHAKEN AS request and feed the generated Identity header in SHAKEN VS request, and get 400/416. Can not find the reasonCode or reasonText in doc.

HTTP/1.1 400 Bad Request
Date: Tue, 24 Jan 2023 21:55:10 GMT
Server: WSGIServer/0.2 CPython/3.7.15
Content-Type: application/json
X-Frame-Options: DENY
Content-Length: 139
X-Content-Type-Options: nosniff
Referrer-Policy: same-origin

{"verifyResults": [{"verifyResult": {"status": "fail", "ppt": "", "reasonCode": 416, "reasonText": "URI parsing failed", "passport": ""}}]}

@rishagit
Copy link
Collaborator

rishagit commented Feb 8, 2023

hi @di-shi
Can you please share the request that you are sending.

@di-shi
Copy link
Author

di-shi commented Feb 8, 2023

@rishagit

A sample request and response is attached except the apiKey is removed.

curl -i -k -s -X POST http://34.233.174.72:8000/stir/v1/verification?apiKey=XXXXXX -H Content-Type: application/json -H Accept: application/json -d {"identityHeader": "eyJhbGciOiJFUzI1NiIsInBwdCI6InNoYWtlbiIsInR5cCI6InBhc3Nwb3J0IiwieDV1IjoiaHR0cHM6Ly9jYS1jbXMubmV1c3RhcmxhYi5iaXovY2NpZC9hdXRobi92Mi9jZXJ0cy8xMTA0MS4xMDAzNyJ9.eyJhdHRlc3QiOiJBIiwiZGVzdCI6eyJ0biI6WyIxNDA0NTI2NjA2MCJdfSwiaWF0IjoxNjc1ODY1MTExLCJvcmlnIjp7InRuIjoiMTU1NTEyMzQ1NjcifSwib3JpZ2lkIjoiOThkMDk2MzktZmJhZi0xMWU3LTlmNTQtMDAwYzI5YjFiMzllIn0.NSh_mvXC322sDwT4BGoi0cUoxxmf35EiNd2edaqmaJNhiaL1J7NUibt-YiXaaPN7Nx5S9baDijQ7JxstjdwU1Q;info=<https://ca-cms.neustarlab.biz/ccid/authn/v2/certs/11041.10037>;alg=ES256;ppt=\"shaken\"", "place": "holder", "to": {"tn": ["sip:[email protected]"]}, "time": 1675865111, "from": {"tn": "sip:[email protected]"}}

HTTP/1.1 400 Bad Request
Date: Wed, 08 Feb 2023 14:05:11 GMT
Server: WSGIServer/0.2 CPython/3.7.15
Content-Type: application/json
X-Frame-Options: DENY
Content-Length: 139
X-Content-Type-Options: nosniff
Referrer-Policy: same-origin

{"verifyResults": [{"verifyResult": {"status": "fail", "ppt": "", "reasonCode": 416, "reasonText": "URI parsing failed", "passport": ""}}]}

@kpolitz
Copy link
Contributor

kpolitz commented Feb 9, 2023

I could not reproduce:

{
"verifyResults": [
{
"verifyResult": {
"ppt": "shaken",
"status": "pass",
"validClaims": {
"attest": "A",
"dest": {
"tn": [
"14045266060"
]
},
"iat": 1675962972,
"orig": {
"tn": "15551234567"
},
"origid": "2088448c-718e-4930-be5d-0ec9b25d2f00"
}
}
}
],
"verstatValue": "TN-Validation-Passed"
}

@di-shi
Copy link
Author

di-shi commented Feb 9, 2023

still failed for me just minutes ago.

curl -i -k -s -X POST http://34.233.174.72:8000/stir/v1/verification?apiKey=XXXXXX -H Content-Type: application/json -H Accept: application/json -d {"identityHeader": "eyJhbGciOiJFUzI1NiIsInBwdCI6InNoYWtlbiIsInR5cCI6InBhc3Nwb3J0IiwieDV1IjoiaHR0cHM6Ly9jYS1jbXMubmV1c3RhcmxhYi5iaXovY2NpZC9hdXRobi92Mi9jZXJ0cy8xMTA0MS4xMDAzNyJ9.eyJhdHRlc3QiOiJBIiwiZGVzdCI6eyJ0biI6WyIxNDA0NTI2NjA2MCJdfSwiaWF0IjoxNjc1OTY2ODUwLCJvcmlnIjp7InRuIjoiMTU1NTEyMzQ1NjcifSwib3JpZ2lkIjoiOThkMDk2MzktZmJhZi0xMWU3LTlmNTQtMDAwYzI5YjFiMzllIn0.VKdm-NO1z7Ccw2mUpCx9I95QQykskE-C66E7SlN9zdzgzV1ltKECq11QdW-zw0L0RRvXpnpahz_D1ifX5iK76w;info=<https://ca-cms.neustarlab.biz/ccid/authn/v2/certs/11041.10037>;alg=ES256;ppt=\"shaken\"", "place": "holder", "to": {"tn": ["sip:[email protected]"]}, "time": 1675966850, "from": {"tn": "sip:[email protected]"}}

HTTP/1.1 400 Bad Request
Date: Thu, 09 Feb 2023 18:20:51 GMT
Server: WSGIServer/0.2 CPython/3.7.15
Content-Type: application/json
X-Frame-Options: DENY
Content-Length: 139
X-Content-Type-Options: nosniff
Referrer-Policy: same-origin

{"verifyResults": [{"verifyResult": {"status": "fail", "ppt": "", "reasonCode": 416, "reasonText": "URI parsing failed", "passport": ""}}]}

@kpolitz
Copy link
Contributor

kpolitz commented Feb 9, 2023

Is this the correct PASSporT information?

{
"alg": "ES256",
"ppt": "shaken",
"typ": "passport",
"x5u": "https://ca-cms.neustarlab.biz/ccid/authn/v2/certs/11041.10037"
}

{
"attest": "A",
"dest": {
"tn": [
"14045266060"
]
},
"iat": 1675966850,
"orig": {
"tn": "15551234567"
},
"origid": "98d09639-fbaf-11e7-9f54-000c29b1b39e"
}

@kpolitz
Copy link
Contributor

kpolitz commented Feb 9, 2023

what does this parameter do in your VS request? "place": "holder",

@di-shi
Copy link
Author

di-shi commented Feb 9, 2023

> Is this the correct PASSporT information?

I think so.

> what does this parameter do in your VS request? "place": "holder",

We use a shell script for the test. Some use cases need more parameters. If a parameter is not needed, we place a placeholder item in the request.

@kpolitz
Copy link
Contributor

kpolitz commented Feb 9, 2023

We believe this has something to do with the orig/dest/to/from and will continue to investigate. Thanks.

@rishagit
Copy link
Collaborator

As @kpolitz mentioned, this is due to wrong format sent for 'from' field. Currently our system is expecting 'from' as object which contains 'tn' array. We are working to fix this and 'from' would be just an object which will accept 'tn' as string (more like the way you are sending)

In your request if you can just change the 'from' field as below, you will get a response. But we are working on a fix for it and eventually 'from' will just be an object which accepts 'tn' as String (similar to your request).

"from": {
        "tn": [
            "sip:[email protected]"
        ]
    }

@di-shi
Copy link
Author

di-shi commented Feb 10, 2023

@rishagit
Thank you for the information. Please let us know when these tickets are fixed. Then we will continue the test.

@kpolitz
Copy link
Contributor

kpolitz commented Feb 16, 2023

"orig" and "from" parameters will no longer be arrays in next pull request. A note will be sent to test participants when change is available.

@kpolitz
Copy link
Contributor

kpolitz commented Mar 13, 2023

Changes were deployed on 2/23/23. Please test and confirm.

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

3 participants