[JAVA-2308] Driver version check fails when there is another version.properties file in the classpath Created: 13/Sep/16 Updated: 26/Dec/19 Resolved: 14/Sep/16 |
|
| Status: | Closed |
| Project: | Java Driver |
| Component/s: | Connection Management |
| Affects Version/s: | None |
| Fix Version/s: | 3.4.0 |
| Type: | Bug | Priority: | Major - P3 |
| Reporter: | Vladimir Dolzhenko | Assignee: | Jeffrey Yemin |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||
| Description |
|
Version: 3.4.0.SNAPSHOT
but there is another "version.properties" file in classpath - it leads to IllegalArgumentException("Value can not be null") in BsonString constructor. This prevents any connections from being established to the MongoDB server. |
| Comments |
| Comment by Githook User [ 15/Sep/16 ] | |
|
Author: {u'username': u'jyemin', u'name': u'Jeff Yemin', u'email': u'jeff.yemin@10gen.com'}Message: Fixes issue where the manifest can not be loaded by older version of Java | |
| Comment by Githook User [ 15/Sep/16 ] | |
|
Author: {u'username': u'jyemin', u'name': u'Jeff Yemin', u'email': u'jeff.yemin@10gen.com'}Message: | |
| Comment by Githook User [ 14/Sep/16 ] | |
|
Author: {u'username': u'jyemin', u'name': u'Jeff Yemin', u'email': u'jeff.yemin@10gen.com'}Message: | |
| Comment by Githook User [ 14/Sep/16 ] | |
|
Author: {u'username': u'jyemin', u'name': u'Jeff Yemin', u'email': u'jeff.yemin@10gen.com'}Message: | |
| Comment by Githook User [ 14/Sep/16 ] | |
|
Author: {u'name': u'Vladimir Dolzhenko', u'email': u'vladimir.dolzhenko@markit.com'}Message: | |
| Comment by Vladimir Dolzhenko [ 13/Sep/16 ] | |
|
Thanks @ross.lawley - I proposed another - much proper fix for the issue | |
| Comment by Ross Lawley [ 13/Sep/16 ] | |
|
Hi vladimir.dolzhenko@ihsmarkit.com, Many thanks for the ticket and the pull request. The client handshake code is still in active development. We'll review your suggested fix and update this ticket once its completed. Ross | |
| Comment by Vladimir Dolzhenko [ 13/Sep/16 ] | |
|
the proper solution has to read value from MANIFEST.MF - but the current manifest doesn't have a version so - smth like
but it means that build process have to write Version attribute to manifest | |
| Comment by Vladimir Dolzhenko [ 13/Sep/16 ] | |
|
git pull request |