-
Notifications
You must be signed in to change notification settings - Fork 41
Issue #78 update PG drivers #85
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
base: main
Are you sure you want to change the base?
Conversation
|
I think I would not bother updating the driver for simplesender. I didn't even realize what that code did until I looked at it. It hasn't been touched in 13 years. That library was added 18 years ago. My best guess is that it was part of some undocumented test process. I don't believe it was included in the mirth distributions, and it doesn't look like it even gets built as part of the normal build process. We probably need to evaluate to see if we want to get rid of it, and if we decide to keep it with the existing postgres functionality, it should be using a modern driver. |
jonbartels
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do not update the JARS in simplesender. They are not packaged in the OIE server or client.
|
@tonygermano - I agree with you. I filed #88 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Copilot wasn't able to review any files in this pull request.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
|
Ignore me asking CoPilot to review please. |
5c39642 to
2256d9f
Compare
39ac634
Signed-off-by: Richard Ogin <rogin@users.noreply.github.com>
Signed-off-by: Tony Germano <tony@germano.name>
39ac634 to
5bde40d
Compare
|
@jonbartels is picking this up again - We've likely let it sit so long the PG drivers are outdated. It definitely needs a rebase against main |
I actually just rebased it less than 12 hours before you did, but the drivers are definitely outdated. |
Issue #78
Pulled latest v42.6.x from here. SHA1 hash verified. Sonatype showed it having zero vulnerabilities.
Could not find other PG jar at version "-405". Pulled latest "-408" version here. 2007 timestamp. SHA1 hash verified.
Code compiled w/exception of KeyEncryptorTest, which I deleted. All failed tests ignored.
Ignore the tool listing two renames - two files were deleted, and two files were added.