-
Notifications
You must be signed in to change notification settings - Fork 163
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
Please add cookie security #38
Comments
@bhoriuchi Would you please submit a pull request? |
PR Created #39 |
@bhoriuchi I only merged PR 37 and I noticed that PR 39 was gone. I can't really merge PR 40 since it's not showing up in PR list and also if I do directly to the link #40 I don't see anything to merge. Can you resubmit the PR again or create a new PR and resubmit? Can you please add a test case with the PR? Thanks! |
@bhoriuchi Thank you for the PR. Can you please include a test case in the PR? |
@rashmihunt test added |
Fixes via #44 |
Working with the vSphere API which requires cookie security
The following works for me, but it would be a nice enhancement to have CookieSecurity added to the ones supported by the project.
The text was updated successfully, but these errors were encountered: