-
Notifications
You must be signed in to change notification settings - Fork 232
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
Regex in response header seems not to work with V3 specification #433
Regex in response header seems not to work with V3 specification #433
Conversation
The log:
|
Thanks. I think raising this is a PR is slightly confusing, as you're not looking to incorporate this change in the code right? It is helpful for a repro for an issue though, so perhaps we ought to convert this to an issue? |
This is the contract generated with V2, which works:
And this is the contract generated with V3:
|
They only differ in the way to describe the matching rules V2:
V3:
|
If I manually remove the "[0]" from the V3 contract then it gets correctly verified. So this also works:
|
I'll need my dear friend @uglyog to help me with this one. Does this look like a bug, and if so, should we move it to pact-reference? |
Looks like a bug in the pact_models crate |
Created an upstream issue to track this here: pact-foundation/pact-reference#238 |
This is a duplicate of #420 I think. |
Closing as we're not looking to merge this. The issue is covered under #420 |
Hi :)
I updated the pact version to 3 and now the issue seems to reproduce.
I'll post now the log with trace level.