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

mcf: highlighting username/clientId issue in DarwinClientRouteBuilder… #2

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mikecflynn
Copy link

….java; added server.port option to application.properties; added .gitignore;

….java; added server.port option to application.properties; added .gitignore;
@mikecflynn
Copy link
Author

I don't whether this issue is a bug or whether I've misunderstood something. The original code must work in some respect otherwise why is it coded so? When I run the original code the 'username' variable in DarwinClientRouteBuilder.java resolves to null. It does resolve in the 'configure' method and so my 'fix' allows me to run the code and receive messages.

I also added a commented out server.port entry in application.properties and a .gitignore so I'm able to test easier. Being a newbie with github and not quite sure the best ways to use it, I had also previously added the properties file with the username/password details to .gitignore but ended up starting again having had difficult getting this to work out properly. Hence you may have seen a previous pull request which I subsequently pulled.

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

Successfully merging this pull request may close these issues.

None yet

2 participants