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]