[GitHub] [incubator-linkis] casionone commented on a diff in pull request #1989: [fix #1761]command "ip addr" maybe return multiple ips

[GitBox <gi...@apache.org>]

casionone commented on code in PR #1989:
URL: https://github.com/apache/incubator-linkis/pull/1989#discussion_r854219679


##########
web/config.sh:
##########
@@ -21,4 +21,4 @@ linkis_port="8088"
 linkis_url="http://localhost:20401"
 
 #linkis ip address
-linkis_ipaddr=$(ip addr | awk '/^[0-9]+: / {}; /inet.*global/ {print gensub(/(.*)\/(.*)/, "\\1", "g", $2)}')
+linkis_ipaddr=localhost

Review Comment:
   the service of linkis-mg-gateway default port is 9001
   I think it should be consistent
   ```
   #URL of the backend service of linkis-mg-gateway
   linkis_url="http://127.0.0.1:9001"
   
   #linkis ip address，the htpp access address for linkis-web will be http://${linkis_ipaddr}:${linkis_port}
   linkis_ipaddr=127.0.0.1
   
   ```



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@linkis.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@linkis.apache.org
For additional commands, e-mail: commits-help@linkis.apache.org