Browse Source

!345 fix bugs of --url-path-prefix parameter [r0.5]

Merge pull request !345 from liangyongxiong/r0.5
tags/v0.5.0-beta
mindspore-ci-bot Gitee 5 years ago
parent
commit
97e980ae34
3 changed files with 15 additions and 5 deletions
  1. +2
    -0
      .gitignore
  2. +12
    -4
      mindinsight/scripts/start.py
  3. +1
    -1
      mindinsight/ui/public/index.html

+ 2
- 0
.gitignore View File

@@ -86,3 +86,5 @@ build/*

output/
!output/README.md

mindinsight/ui/public/static/js/graphvizlib.wasm

+ 12
- 4
mindinsight/scripts/start.py View File

@@ -122,7 +122,8 @@ class PortAction(argparse.Action):
class UrlPathPrefixAction(argparse.Action):
"""Url Path prefix action class definition."""

REGEX = r'^(\/[a-zA-Z0-9-\-\.]+)+$'
INVALID_SEGMENTS = ('.', '..')
REGEX = r'^[a-zA-Z0-9_\-\.]+$'

def __call__(self, parser, namespace, values, option_string=None):
"""
@@ -135,8 +136,12 @@ class UrlPathPrefixAction(argparse.Action):
option_string (str): Optional string for specific argument name. Default: None.
"""
prefix = values
if not re.match(self.REGEX, prefix):
parser.error(f'{option_string} value is invalid url path prefix')
segments = prefix.split('/')
for index, segment in enumerate(segments):
if not segment and index in (0, len(segments) - 1):
continue
if segment in self.INVALID_SEGMENTS or not re.match(self.REGEX, segment):
parser.error(f'{option_string} value is invalid url path prefix')

setattr(namespace, self.dest, prefix)

@@ -186,7 +191,10 @@ class Command(BaseCommand):
type=str,
action=UrlPathPrefixAction,
help="""
Custom path prefix for web page address. Default value is ''.
Custom URL path prefix for web page address. URL path prefix
consists of segments separated by slashes. Each segment supports
alphabets / digits / underscores / dashes / dots, but cannot just
be emtpy string / single dot / double dots. Default value is ''.
""")

for hook in HookUtils.instance().hooks():


+ 1
- 1
mindinsight/ui/public/index.html View File

@@ -21,7 +21,7 @@ limitations under the License.
<meta charset="utf-8" />
<meta http-equiv="X-UA-Compatible" content="IE=edge" />
<meta name="viewport" content="width=device-width,initial-scale=1.0" />
<link rel="icon" href="<%= BASE_URL %>/static/img/favicon.png" />
<link rel="icon" href="static/img/favicon.png" />
<title>MindInsight</title>
<style>
.errorInfo {


Loading…
Cancel
Save