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

fix: always close file descriptors after parsing LDIF files #159

Merged
merged 1 commit into from
Dec 23, 2021

Conversation

iromli
Copy link
Contributor

@iromli iromli commented Dec 10, 2021

When parsing LDIF files, scripts will ensure file descriptors are closed using with context. Related issue: #153

@iromli iromli requested a review from moabu as a code owner December 10, 2021 16:03
@iromli iromli linked an issue Dec 10, 2021 that may be closed by this pull request
@moabu moabu added 1.0 Address in branch 1.0 kind-bug Issue or PR is a bug in existing functionality comp-docker-jans-persistence-loader labels Dec 23, 2021
@moabu moabu added this to the 1.0.0 milestone Dec 23, 2021
@moabu moabu merged commit a99bef8 into master Dec 23, 2021
@moabu moabu deleted the cn-ldif-parser branch December 23, 2021 11:24
moabu pushed a commit that referenced this pull request Jan 13, 2022
…2fb8048871c9a00e

[Snyk] Fix for 1 vulnerabilities
moabu pushed a commit that referenced this pull request Jan 13, 2022
Unit Tests: fixing the problem: invalid length of ec key attributes (length > 32): issue oxauth.#159
moabu pushed a commit that referenced this pull request Jan 13, 2022
Unit Tests: fixing the problem: invalid length of ec key attributes (length > 32): issue oxauth.#159
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1.0 Address in branch 1.0 comp-docker-jans-persistence-loader kind-bug Issue or PR is a bug in existing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Potential resource leak while parsing LDIF files
2 participants