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.

Sign up for GitHub

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

Jump to bottom

Error in micro-services mode: 400 Client Error #78

Closed
davisjam opened this issue Sep 22, 2018 · 2 comments
Closed

Error in micro-services mode: 400 Client Error #78

davisjam opened this issue Sep 22, 2018 · 2 comments

Comments

@davisjam
Copy link

  1. Start micro-services
Node-DC-EIS/Node-DC-EIS-microservices $ for d in address_service comp_service db_loader_service employee_service family_service health_service photo_service; do echo $d; cd $d; PATH=$FLOSS/node:$PATH NODE_THREADPOOL_TYPE=SPLIT_BY_ORIGIN node server.js & sleep 2; cd -; done
  1. Run client
$ python runspec.py --config config.json

(config.json includes the modified line "server_port":"3000",)

Output begins OK but after loading the DB it prints:

HTTP error occured
400 Client Error: Bad Request for url: http://localhost:3000/employees

(many times).
@davisjam
Copy link
Author

@uttampawar

This was referenced Oct 1, 2018
Open
Closed
@uttampawar
Copy link
Collaborator

@davisjam Let me know if you still see this issue after our discussion last week.

@davisjam davisjam mentioned this issue Feb 6, 2019
Closed
Jonathan-Alexander added a commit to Jonathan-Alexander/Node-DC-EIS that referenced this issue Mar 4, 2019
@Jonathan-Alexander
Fix: microservices Bad URL Error
98aee0d 
When running in microservices mode the client receives the error
400: Bad URL localhost:3000/employees

This looks to be caused by posts to addNewEmployee being made without
a last name for the employee.

Add logic similar to loader.js from monolithic code to db_loader_service/controllers/loader.js
to ensure enough last names for all the users to be added.

Fixes: Node-DC#78
@Jonathan-Alexander Jonathan-Alexander mentioned this issue Mar 4, 2019
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@uttampawar @davisjam