[
https://issues.apache.org/jira/browse/DAYTRADER-51?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Forrest Xia closed DAYTRADER-51.
--------------------------------
Resolution: Fixed
Fix Version/s: 3.0
3.0-M2
2.2.1
2.1.7
Committed to 2.1 branch at revision 1038645
Committed to 2.2 branch at revision 1038647.
Committed to trunk at revision 1038649.
> NPE when logging in with an unknown username
> --------------------------------------------
>
> Key: DAYTRADER-51
> URL: https://issues.apache.org/jira/browse/DAYTRADER-51
> Project: DayTrader
> Issue Type: Bug
> Components: Web Tier
> Affects Versions: 2.0
> Reporter: Jarek Gawor
> Assignee: Forrest Xia
> Fix For: 2.1.7, 2.2.1, 3.0-M2, 3.0
>
>
> I was testing DayTrayder (in web services mode) and tried to log in with a
> bad username. That resulted in a bad NPE exception because
> org.apache.geronimo.samples.daytrader.direct.TradeDirect.login() returned
> null instead of throwing an exception. I assume an exception should be
> raised. If null can be returned the web services code needs to be changed to
> handle that.
> Here's what I see in the logs:
> 14:29:43,775 ERROR [Log] Error: TradeDirect:login -- failure to find account
> forhhh
> 14:29:43,776 ERROR [Log] Error: TradeDirect:login -- error logging in user
> javax.ejb.FinderException: Cannot find account forhhh
> javax.ejb.FinderException: Cannot find account forhhh
> at
> org.apache.geronimo.samples.daytrader.direct.TradeDirect.login(TradeDirect.java:1487)
> at
> org.apache.geronimo.samples.daytrader.TradeAction.login(TradeAction.java:455)
> at
> org.apache.geronimo.samples.daytrader.soap.TradeWSAction.login(TradeWSAction.java:177)
> .....
> Caused by: java.lang.NullPointerException
> at
> org.apache.geronimo.samples.daytrader.soap.Convert.convertAccountDataBean(Convert.java:97)
> at
> org.apache.geronimo.samples.daytrader.soap.TradeWSAction.login(TradeWSAction.java:177)
> ... 41 more
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
