fix cors and requestData errors #59
Conversation
| return [ | ||
| { | ||
| source: '/api/v1/:path*', // Match all routes starting with /api/v1/ | ||
| destination: 'http://localhost:8080/api/v1/:path*', // Proxy to backend server |
There was a problem hiding this comment.
I don't think the http://localhost:8080 prefix should be hardcoded. Is it possible to just use the remaining suffix?
Granted, I don't know next.js, but this seems off.
There was a problem hiding this comment.
This is how the nextJS rewrites to an external server work. If I try to replace the hardcoded url with an env variable then I get a 500 internal server error.
Here is the rewrite docs: https://nextjs.org/docs/pages/api-reference/config/next-config-js/rewrites#rewriting-to-an-external-url
There was a problem hiding this comment.
Looks fine to me now.
Tested it with the container PR, and the frontend works and pulls the javascript sources.
| }; | ||
|
|
||
| try { | ||
| const response = await axios.post( | ||
| 'http://localhost:9000/api/tx/programmable-token/issue', | ||
| '/api/v1/tx/programmable-token/issue', |
There was a problem hiding this comment.
this looks more like a route I would expect.
There was a problem hiding this comment.
Yes, this is the direct api call not the rewrite.
|
As mentioned on slack, the solution of the conflict is to still delete |
-reapplied the rewrites to the next.config.js file
-fixed the issues with the requestData