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

com.aol.micro.server.auto.discovery.Rest doesn't work #18

Closed
mwnorman opened this issue Apr 2, 2015 · 5 comments
Closed

com.aol.micro.server.auto.discovery.Rest doesn't work #18

mwnorman opened this issue Apr 2, 2015 · 5 comments

Comments

@mwnorman
Copy link

mwnorman commented Apr 2, 2015

ref my gist (https://gist.github.com/mwnorman/ed36b3c772d1973dcf1a)
I have created a simple test app ... my HelloResource is not being auto-magically discovered:

http://localhost:8080/simple/foo/hello
Resource identified by path '/simple/foo/hello', does not exist.

Am I missing something?

@johnmcclean
Copy link
Member

I'm guessing this is also dependency related. I'm upgrading to the latest Gradle. Gradle not including dependencies in the generate POM was an issue, but it's marked as resolved in their JIRA.

I can replicate the issue locally with your POM, will try and determine the exact cause.

@johnmcclean
Copy link
Member

This is a bug. As a temporary work around you can add the @microserver annotation to TestMicroserverApp it will work.
Failing unit test added & fixed locally.

johnmcclean added a commit that referenced this issue Apr 2, 2015
@johnmcclean
Copy link
Member

Tested locally, and working with v0.54 which is in Maven central now.

@kewangie
Copy link
Contributor

kewangie commented Apr 3, 2015

Tested with v0.53. I believe that HelloResource needs to be put into a package where spring scans for annotated components

@johnmcclean
Copy link
Member

Yep, the names of the packages Spring should scan weren't being configured correctly - when no Microserver annotation of Config object present. Fixed now in v0.54.

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

No branches or pull requests

3 participants