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

Handle missing manifest files in JarTypeFilter #24597

Closed
wants to merge 1 commit into from

Conversation

edwardsre
Copy link
Contributor

JarTypeFilter was not handling dependency jars with missing manifest files, resulting in all starter jars being included in repackaged jars.

Fixes gh-24594

JarTypeFilter was not handling dependency jars with missing manifest files, resulting in all starter jars being included in repackaged jars.

Fixes spring-projectsgh-24594
@pivotal-issuemaster
Copy link

@edwardsre Please sign the Contributor License Agreement!

Click here to manually synchronize the status of this Pull Request.

See the FAQ for frequently asked questions.

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Dec 22, 2020
@pivotal-issuemaster
Copy link

@edwardsre Thank you for signing the Contributor License Agreement!

@snicoll snicoll added type: bug A general bug and removed status: waiting-for-triage An issue we've not yet triaged labels Dec 23, 2020
@snicoll snicoll added this to the 2.4.x milestone Dec 23, 2020
@snicoll snicoll assigned snicoll and unassigned snicoll Dec 23, 2020
@snicoll snicoll modified the milestones: 2.4.x, 2.4.2 Dec 23, 2020
snicoll pushed a commit that referenced this pull request Dec 23, 2020
This commit makes sure that a jar file without a manifest is handled
properly (i.e. included in the repackaged archive).

See gh-24597
@snicoll snicoll closed this in 5fa5b62 Dec 23, 2020
@snicoll
Copy link
Member

snicoll commented Dec 23, 2020

@edwardsre thank you for making your first contribution to Spring Boot.

@edwardsre edwardsre deleted the gh-24594 branch December 23, 2020 16:48
snicoll added a commit that referenced this pull request Jan 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug A general bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants