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 6ea11f4 to f8fab66 Compare February 7, 2020 12:25
@TomGranot TomGranot added this to Triage in Triage Mar 17, 2021
@TomGranot TomGranot moved this from Triage to In progress in Triage Mar 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Triage
  
In progress
Development

Successfully merging this pull request may close these issues.

None yet

2 participants