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

Modify nginx conf in AWS EC2 #76

Closed
yunchae-kim opened this issue Sep 20, 2023 · 0 comments
Closed

Modify nginx conf in AWS EC2 #76

yunchae-kim opened this issue Sep 20, 2023 · 0 comments
Assignees
@yunchae-kim
Labels
minor Easy to fix issue that can be completed in a limited amount of time, similar to "easy fix"

Comments

@yunchae-kim
Copy link
Contributor

yunchae-kim commented Sep 20, 2023 •
edited
Loading

The current system of Comptox AI website is structured to host the webpage in AWS EC2, but the data-related part are stored in the university server.

The conf file in EC2 must be modified so that the redirection to the servers reflects this division of servers when tasks are called.

TODO:

  • Copy conf file from Comptox AI ssh
  • Modify redirection route of copied conf file
  • Store the modified conf file to EC2
@yunchae-kim yunchae-kim added the minor Easy to fix issue that can be completed in a limited amount of time, similar to "easy fix" label Sep 20, 2023
@yunchae-kim yunchae-kim self-assigned this Sep 20, 2023
yunchae-kim added a commit that referenced this issue Sep 28, 2023
@yunchae-kim
fix: Change 'api/app.js' redirection to 'localhost:3000/api'
f002f73 
[#76] Refer to GitHub issue...

This update addresses the following:

- Corrected the redirection URL from 'http://localhost:3000' to
'http://localhost:3000/api' to ensure that address redirection functions
correctly. With this change,
'http://ec2-<ip>.compute-1.amazonaws.com/api' will correctly redirect to
the API page as intended.

Additionally, this update includes the following changes:

- Reformatted constant imports to use 'import' to align with the ES6
style guide and reordered them based on the import order standard.
- Replaced some double quotations with single quotations to maintain
consistent coding style.
- Replaced certain 'req' variables used in functions with '_' to resolve
ESLint issues related to unused variables.
- Updated some functions to use ES6 arrow function syntax for improved
code readability.

These changes enhance code clarity and maintain consistency with coding
standards.
@yunchae-kim yunchae-kim mentioned this issue Sep 28, 2023
Merged
yunchae-kim added a commit that referenced this issue Sep 28, 2023
@yunchae-kim
fix: Change redirection link of HOST to 'localhost:3000/api'
422da8e 
[#76] Refer to GitHub issue...

This update changes the HOST variable to redirect api to
localhost:3000/api instead of localhost:3000.
The change was intended to be done in PR-#81 but omitted by mistake,
which is fixed in this PR.
yunchae-kim added a commit that referenced this issue Sep 30, 2023
@yunchae-kim
Revert "fix: Change 'api/app.js' redirection to 'localhost:3000/api'"
cb0a023 
This reverts commit f002f73.

revert: Revert commit f002f73

[#76] Refer to GitHub issue...

This commit revert is intended to debug nginx api redirection.
All changes to api/app.js will be reverted.
@yunchae-kim yunchae-kim mentioned this issue Sep 30, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@yunchae-kim yunchae-kim

Labels
minor Easy to fix issue that can be completed in a limited amount of time, similar to "easy fix"
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@yunchae-kim