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

#1165 - filter expired cookies when injected in httpClient #1314

Merged
merged 2 commits into from
Oct 10, 2020
Merged

#1165 - filter expired cookies when injected in httpClient #1314

merged 2 commits into from
Oct 10, 2020

Conversation

chaudharydeepak
Copy link
Contributor

#1165 - filter expired cookies when injected in httpClient

Includes fixes to remove expired cookies using httpClient. Fix is implemented both for jersey as well as apache clients.

@chaudharydeepak
Copy link
Contributor Author

@ptrthomas - opened this PR for #1165 - filter expired cookies when injected in httpClient. Let me know for any feedback. Thank you.

@ptrthomas ptrthomas merged commit 038802f into karatelabs:develop Oct 10, 2020
@ptrthomas
Copy link
Member

@chaudharydeepak this is good work ! thanks 👍

@ptrthomas
Copy link
Member

@chaudharydeepak in case you have used cookies for UI testing in karate - do take a look at this one: #1293

@chaudharydeepak
Copy link
Contributor Author

chaudharydeepak commented Oct 10, 2020

@ptrthomas Thank you! I am not too familiar with UI testing - though I am definitely interested in workimg on this - I can figure it out & will ask questions if I have any. If its ok, please assign #1293 to me & I will get start.

@ptrthomas
Copy link
Member

@chaudharydeepak great, thanks - I also saw this other one if you prefer that: #1211

either way, for some reason I can't assign it to you until you make a comment on the issue directly

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

Successfully merging this pull request may close these issues.

2 participants