Alex Pavlovich

When writing a class in javascript, the expected behavior is that when you write some code like: ```js function My() { this.value = arguments[0] || 1; } My.prototype.method = function()...

NullPointerException when mapping is null (such as when used with spring data cassandra repositories)

2

"mapping" can be null here, causing the `.isExported()` to throw a NPE. https://github.com/spring-projects/spring-data-rest/blob/a40aec5debf496b8e29d66a720ad27989b04f2bd/spring-data-rest-webmvc/src/main/java/org/springframework/data/rest/webmvc/ProfileController.java#L94-L111 I thought i was using the strategy to only include annotated repositories and that would not happen...

Consider removing request (as it is deprecated)

1

Opening this as I don't see it is mentioned in jsdom issue 2792 but not in this library yet.

This should say static instead of contentBase https://github.com/rwieruch/minimal-webpack-setup/blob/a3bd10be76feab91f2fb4bc82777417d4c308e9b/webpack.config.js#L10 per https://gist.github.com/johnrichardrinehart/c8ec6ab1e60f39fc3b8dc738db649ec0#gistcomment-3671351

Upgrade test deps junit/hamcrest (removes CVE warning on Maven Central)

1

Thanks for considering this pull request, it upgrades junit 4 to 5 and includes separate hamcrest matchers. I would appreciate a new version published, (say 1.1.2) after/if pr is accepted....