Bug 1039591

Summary: "Share this link" for creating new application includes suffix of manifest file in default domain name
Product: OpenShift Online Reporter: gideon
Component: Management ConsoleAssignee: Jordan Liggitt <jliggitt>
Status: CLOSED CURRENTRELEASE QA Contact: libra bugs <libra-bugs>
Severity: low Docs Contact:
Priority: low    
Version: 2.xCC: jliggitt, yujzhang
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
: 1049741 (view as bug list) Environment:
Last Closed: 2014-01-30 00:52:45 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Bug Depends On:    
Bug Blocks: 1049741    

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.