Ylisen opened a new pull request #4688:
URL: https://github.com/apache/incubator-dolphinscheduler/pull/4688


   Create a new project the name contain "http",If you enter the project 
process list, an Error 404 will be reported,and the interface of the new 
process will also report an Error 404, and the URL path of the front-end 
splicing is not correct
   
   
   this closes #4640 
   ## Brief change log
   before change
   ```
   const resolveURL = (url) => {
     if (url.indexOf('http')  !== -1) {
       return url
     }
     if (url.charAt(0) !== '/') {
       return `${apiPrefix}/${url.replace(reSlashPrefix, '')}`
     }
   
     return url
   }
   ```
   
   after change
   ```
   const resolveURL = (url) => {
     if (url.indexOf('http')  === 0) {
       return url
     }
     if (url.charAt(0) !== '/') {
       return `${apiPrefix}/${url.replace(reSlashPrefix, '')}`
     }
   
     return url
   }
   ```
   
   
   ## Verify this pull request
   
   *(Please pick either of the following options)*
   
   This pull request is code cleanup without any test coverage.
   
   *(or)*
   
   This pull request is already covered by existing tests, such as *(please 
describe tests)*.
   
   (or)
   
   This change added tests and can be verified as follows:
   
   *(example:)*
   
     - *Added dolphinscheduler-dao tests for end-to-end.*
     - *Added CronUtilsTest to verify the change.*
     - *Manually verified the change by testing locally.*
   


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to