gered/clj-htmltopdf
1
0
Fork 0
Code Issues Pull requests Packages Projects Releases Wiki Activity

Mitigate Vulnerabilities: CVE-2022-36033, CVE-2021-37714, CVE-2021-29425 #8

Closed
Luke1298 wants to merge 5 commits from master into master
pull from: master
merge into: gered:master
Conversation 2 Commits 5 Files changed 5 +27 -8
Luke1298 commented 2023-01-24 17:09:57 -05:00 (Migrated from github.com)
Copy link

CVE's conveniently described here: https://mvnrepository.com/artifact/clj-htmltopdf/clj-htmltopdf/0.2

CVE's conveniently described here: https://mvnrepository.com/artifact/clj-htmltopdf/clj-htmltopdf/0.2
gered commented 2023-01-26 14:39:03 -05:00 (Migrated from github.com)
Copy link

Thanks! I'll get to this a bit later today and push out a new version.

Thanks! I'll get to this a bit later today and push out a new version.
gered commented 2023-01-29 21:35:31 -05:00 (Migrated from github.com)
Copy link

I had to clean out some work-in-progress stuff that I had originally committed to the "master" branch a long time ago, and then never came back to. It was not ready to be included in a new version, and realistically I should've put those changes on a separate branch to begin with (not sure why I didn't).

Thusly, I merged this change of yours in a bit of a roundabout way because, truthfully, I have no idea if Github's PR functionality could've done it easier (I'm a bit of a Github Pull Request "noob" :-P )

I'm closing this PR because it's already been merged, but Github just doesn't know it (probably because of the other old changes of mine that I backed out).

Thanks! New version [clj-htmltopdf "0.2.1"] is deployed to Clojars with this change included.

I had to clean out some work-in-progress stuff that I had originally committed to the "master" branch a long time ago, and then never came back to. It was not ready to be included in a new version, and realistically I should've put those changes on a separate branch to begin with (not sure why I didn't). Thusly, I merged this change of yours in a bit of a roundabout way because, truthfully, I have no idea if Github's PR functionality could've done it easier (I'm a bit of a Github Pull Request "noob" :-P ) I'm closing this PR because it's already been merged, but Github just doesn't know it (probably because of the other old changes of mine that I backed out). Thanks! New version `[clj-htmltopdf "0.2.1"]` is deployed to Clojars with this change included.
👍 1

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
bug

   
duplicate

   
enhancement

   
help wanted

   
invalid

   
question

   
wontfix
No labels
bug
duplicate
enhancement
help wanted
invalid
question
wontfix
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: gered/clj-htmltopdf#8
No description provided.
Delete branch "master"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?