jhipster / generator-jhipster

JHipster is a development platform to quickly generate, develop, & deploy modern web applications & microservice architectures.
https://www.jhipster.tech
Apache License 2.0
21.47k stars 4.02k forks source link

Wrong repository in MongoDBTokenStore.removeRefreshToken #3436

Closed mwhitis closed 8 years ago

mwhitis commented 8 years ago
Overview of the issue

In the Generated config for MongoDBTokenStore for a Mongo/Oauth app, the removeRefreshToken method is referencing the wrong repository, so throws an error on build.

line 88 in my generated version is:

oAuth2AccessTokenRepository.delete(refreshToken);

should be

oAuth2RefreshTokenRepository.delete(refreshToken);

Motivation for or Use Case

Doesn't build.

JHipster Version(s)

New build today (4/21/16) with 3.1.0. This is my first Mongo/Oauth build, so can't say if it's a regression.

JHipster configuration, a .yo-rc.json file generated in the root folder

{ "generator-jhipster": { "jhipsterVersion": "3.1.0", "baseName": "servicenow_api", "packageName": "com.ge.ui", "packageFolder": "com/ge/ui", "serverPort": "8080", "authenticationType": "oauth2", "hibernateCache": "no", "clusteredHttpSession": "no", "websocket": "no", "databaseType": "mongodb", "devDatabaseType": "mongodb", "prodDatabaseType": "mongodb", "searchEngine": "no", "buildTool": "maven", "useSass": false, "applicationType": "monolith", "testFrameworks": [ "gatling" ], "jhiPrefix": "jhi", "enableTranslation": true, "nativeLanguage": "en", "languages": [ "en" ] } }

Browsers and Operating System

Windows 10, JHipster 3.1.0

Reproduce the error

Generate a new app then run a mvn spring-boot:run

Suggest a Fix

oAuth2AccessTokenRepository.delete(refreshToken);

should be

oAuth2RefreshTokenRepository.delete(refreshToken);

The correct repo is already defined in the file, just the wrong repo referenced at this location.

I'm willing to submit a pull request if it will be helpful.

mwhitis commented 8 years ago

Opened pr: #3437