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

Instantiate CmsSigner outside loop #509

Closed
wants to merge 1 commit into from
Closed

Instantiate CmsSigner outside loop #509

wants to merge 1 commit into from

Conversation

swoga
Copy link
Contributor

@swoga swoga commented Sep 23, 2019

Closes #508

to avoid multiple open data readers
@jstedfast
Copy link
Owner

Ah, committed my fix before I noticed you had sent in a PR. Still waking up out of my morning fog, I guess.

Patch looks good, though! :-)

@jstedfast jstedfast closed this Sep 23, 2019
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.

GetCmsSigner() opens multiple data readers
2 participants