Bug 1039591 - "Share this link" for creating new application includes suffix of manifest file in default domain name
Summary: "Share this link" for creating new application includes suffix of manifest fi...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: OpenShift Online
Classification: Red Hat
Component: Management Console
Version: 2.x
Hardware: Unspecified
OS: Unspecified
low
low
Target Milestone: ---
: ---
Assignee: Jordan Liggitt
QA Contact: libra bugs
URL:
Whiteboard:
Depends On:
Blocks: 1049741
TreeView+ depends on / blocked
 
Reported: 2013-12-09 14:53 UTC by gideon
Modified: 2015-05-15 01:36 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
: 1049741 (view as bug list)
Environment:
Last Closed: 2014-01-30 00:52:45 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)

Description gideon 2013-12-09 14:53:52 UTC
Description of problem:

"Share this link" for creating new applications includes the suffix of the manifest file in the default URL.


How reproducible:

Always


Steps to Reproduce:
1. Click the "Add application..." button in the web console.
2. Enter the URL to a manifest e.g. https://raw.github.com/accursoft/Haskell-Cloud/master/.openshift/manifests/network.yml.
2. Click "Next"
3. Click "Share this link"

Actual results:

The "Public URL" field has defaulted to networkyml.


Expected results:

It should be empty or network.

Comment 1 Jordan Liggitt 2013-12-10 16:21:13 UTC
Will merge in https://github.com/openshift/origin-server/pull/4304

Comment 2 Jordan Liggitt 2013-12-10 16:22:06 UTC
Will suggest names from custom cart manifest hashes (http://foo.bar#custom_name) or name params (http://foo.bar?name=custom_name), but not from the manifest url or path

Comment 3 openshift-github-bot 2013-12-11 17:22:16 UTC
Commit pushed to master at https://github.com/openshift/origin-server

https://github.com/openshift/origin-server/commit/7928aac8bb4dbdb123cd457442f803471a2fcc99
Fix bug 1039591: Don't use url basename as suggested app name

Comment 4 Yujie Zhang 2013-12-12 05:44:29 UTC
Tested on devenv_4125, the suggested app name is not "manifestyml" now after clicking the “share this link” link, so verify this bug, thanks.


Note You need to log in before you can comment on or make changes to this bug.