fix #12592

[chavacava]

fix #12592 by changing a by-value method receiver into a pointer receiver

[tgross]

Thanks for catching this @chavacava!

I took a look at where this was introduced (ref 6a6547b) and it looks like it the consequence of this bug is that we don't properly reset the blocking timeout when we're retrying blocking queries from the client. This could result in a blocking query running longer than expected. I've added a changelog entry. We'll ship this in Nomad 1.3.0 and backport it to 1.2.x and 1.1.x.

[github-actions]

I'm going to lock this pull request because it has been closed for 120 days ⏳. This helps our maintainers find and focus on the active contributions.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.