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

NPE when getting default admin port #789

Closed
jensstn opened this Issue Aug 31, 2017 · 4 comments

Comments

Projects
None yet
3 participants
@jensstn
Contributor

jensstn commented Aug 31, 2017

core/src/main/java/org/arquillian/cube/impl/client/container/ContainerConfigurationController.java:getDefaultPortFromConfigurationInstance can throw a NullPointerException when calling invoke on the Method returned by fieldName.getReadMethod() (L.105).
This has been observed running against a glassfish-remote-container.

@bartoszmajsak

This comment has been minimized.

Show comment
Hide comment
@bartoszmajsak

bartoszmajsak Aug 31, 2017

Member

Any idea why it throws NPE?

Member

bartoszmajsak commented Aug 31, 2017

Any idea why it throws NPE?

@jensstn

This comment has been minimized.

Show comment
Hide comment
@jensstn

jensstn Sep 1, 2017

Contributor

i didn't have time to dig a lot deeper and it's been a few days, but as far as i remember it happens because it finds a PropertyDescriptor which only has a write method.
do you want me to dig further?

Contributor

jensstn commented Sep 1, 2017

i didn't have time to dig a lot deeper and it's been a few days, but as far as i remember it happens because it finds a PropertyDescriptor which only has a write method.
do you want me to dig further?

@bartoszmajsak

This comment has been minimized.

Show comment
Hide comment
@bartoszmajsak

bartoszmajsak Sep 1, 2017

Member

If you could afford some time for figuring it out that would be awesome. Thanks a lot for your contribution!

Member

bartoszmajsak commented Sep 1, 2017

If you could afford some time for figuring it out that would be awesome. Thanks a lot for your contribution!

@lordofthejars lordofthejars added this to the 1.8.1 milestone Sep 6, 2017

@lordofthejars lordofthejars added the bug label Sep 6, 2017

@lordofthejars

This comment has been minimized.

Show comment
Hide comment
@lordofthejars

lordofthejars Sep 6, 2017

Member

I close the issue because I have merged the PR, but of course if you have time it is still interesting to know why this was null. I just merge the PR so we can release a new version with this fix.

Member

lordofthejars commented Sep 6, 2017

I close the issue because I have merged the PR, but of course if you have time it is still interesting to know why this was null. I just merge the PR so we can release a new version with this fix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment