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

#1094: Honoring updates to realm and proxy realm from ResponseFilter #1311

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

Diagoras
Copy link
Contributor

I can add some tests around this if you want, just say the word.

@slandelle
Copy link
Contributor

Sorry, I don't get what your changes are supposed to fix. Could you elaborate please?
And yes, please add tests that fail without your changes.

@Diagoras
Copy link
Contributor Author

This is an attempt at fixing issue #1094. I'm on vacation for the next week, but will try to add some tests and push up my changes after that.

@Diagoras
Copy link
Contributor Author

@slandelle Actually, I was wondering, if I have random questions when working on this project, what would be the best way to get in contact with you? Are you on IRC, or Gitter?

@slandelle
Copy link
Contributor

@Diagoras I don't use Gitter, and am almost never connected on IRC. Best way is Gtalk (slandelle at gatling dot io).

@slandelle slandelle changed the title #1094: Honoring updates to realm and proxy realm #1094: Honoring updates to realm and proxy realm from ResponseFilter Jan 26, 2017
@slandelle slandelle force-pushed the master branch 5 times, most recently from 9334a56 to 391b4d8 Compare November 28, 2017 12:10
@slandelle slandelle force-pushed the master branch 5 times, most recently from 6ea11f4 to f8fab66 Compare February 7, 2020 12:25
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

Successfully merging this pull request may close these issues.

2 participants