Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 354909 - DefaultBundleReader incorrectly treats Manifest entry keys as case sensitive
Summary: DefaultBundleReader incorrectly treats Manifest entry keys as case sensitive
Status: RESOLVED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: Tycho (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P2 normal (vote)
Target Milestone: ---   Edit
Assignee: Jan Sievers CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-08-17 03:26 EDT by Marc Wirth CLA
Modified: 2021-04-28 16:55 EDT (History)
1 user (show)

See Also:

Attachments
simple patch proposal, not tested, no automatic test. (1.05 KB, application/octet-stream)
2011-08-17 03:27 EDT, Marc Wirth CLA 		no flags Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Marc Wirth CLA 2011-08-17 03:26:48 EDT 
Build Identifier: Tycho 0.12.0

We ran across an build error (compiler could resolve classes) because one used bundle declared a "Bundle-Classpath" (lower-case p), but Tycho treats attribute names in a case-sensitive way while the spec says that attribute names are case insensitive.

The main culprit seems to be the class DefaultBundleReader which converts the main attributes to Properties before looking at the header.

Reproducible: Always

Steps to Reproduce:
1. Create a bundle with an additional Jar on its classpath
2. Use wrong upper/lower-case spelling
3. Try to compile against that bundle where a class of the additional Jar is involved.
Comment 1 Marc Wirth CLA 2011-08-17 03:27:29 EDT 
Created attachment 201621 [details]
simple patch proposal, not tested, no automatic test.
Comment 2 Jan Sievers CLA 2011-10-05 09:16:29 EDT 
fixed with commit 7f57426

we now consistently use class org.eclipse.osgi.framework.util.Headers for parsing MANIFESTs which provides case-insenstive getters for manifest headers.