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

FutureContractChain Invalid value for DayOfMonth #11

Closed
subes opened this issue Sep 27, 2024 · 3 comments · Fixed by #12
Closed

FutureContractChain Invalid value for DayOfMonth #11

subes opened this issue Sep 27, 2024 · 3 comments · Fixed by #12

Comments

@subes
Copy link
Contributor

subes commented Sep 27, 2024

Caused by: net.jacobpeterson.iqfeed4j.util.csv.mapper.CSVMappingException: Error mapping at index 0 with offset 2!
	at net.jacobpeterson.iqfeed4j.util.csv.mapper.list.ListCSVMapper.mapToList(ListCSVMapper.java:70)
	at net.jacobpeterson.iqfeed4j.feed.lookup.optionchains.OptionChainsFeed.handleRequestIDSingleMessageList(OptionChainsFeed.java:424)
	at net.jacobpeterson.iqfeed4j.feed.lookup.optionchains.OptionChainsFeed.onMessageReceived(OptionChainsFeed.java:364)
	at net.jacobpeterson.iqfeed4j.feed.AbstractFeed.run(AbstractFeed.java:184)
	at java.base/java.lang.Thread.run(Thread.java:840)
Caused by: java.time.DateTimeException: Invalid value for DayOfMonth (valid values 1 - 28/31): 0
	at java.base/java.time.temporal.ValueRange.checkValidValue(ValueRange.java:319)
	at java.base/java.time.temporal.ChronoField.checkValidValue(ChronoField.java:718)
	at java.base/java.time.LocalDate.of(LocalDate.java:252)
	at net.jacobpeterson.iqfeed4j.feed.lookup.optionchains.OptionChainsFeed$CSVPOJOPopulators.futureContract(OptionChainsFeed.java:204)
	at net.jacobpeterson.iqfeed4j.util.csv.mapper.list.ListCSVMapper.mapToList(ListCSVMapper.java:67)
	... 4 more

net.jacobpeterson.iqfeed4j.feed.lookup.optionchains.OptionChainsFeed.CSVPOJOPopulators.futureContract(FutureContract, String)

DayOfMonth should be 1, not 0 there.

subes added a commit to subes/IQFeed4j that referenced this issue Sep 27, 2024
use 1 instead of 0 for day
@subes
Copy link
Contributor Author

subes commented Sep 27, 2024

Pull Request: #12

@Petersoj Petersoj linked a pull request Sep 27, 2024 that will close this issue
Petersoj added a commit that referenced this issue Sep 27, 2024
@Petersoj
Copy link
Owner

Will publish patch release soon.

@Petersoj
Copy link
Owner

Fixed in 6.2-2.0.0. Thanks for contributing!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants