Bug 827337 - The url of two applications with name "kit" and "Kit" will redirected to the same page.
Summary: The url of two applications with name "kit" and "Kit" will redirected to the ...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: OKD
Classification: Red Hat
Component: Pod
Version: 2.x
Hardware: Unspecified
OS: Unspecified
medium
low
Target Milestone: ---
: ---
Assignee: Abhishek Gupta
QA Contact: libra bugs
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-06-01 07:22 UTC by Yujie Zhang
Modified: 2015-05-15 01:57 UTC (History)
8 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-06-25 18:26:35 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
screenshot1 (47.20 KB, image/png)
2012-06-01 07:22 UTC, Yujie Zhang
no flags Details

Description Yujie Zhang 2012-06-01 07:22:56 UTC
Created attachment 588345 [details]
screenshot1

Description of problem:

I created one app named "kit" using Kitchensink template, and another app named "Kit" of nodejs type.

Version-Release number of selected component (if applicable):

int.openshift.redhat.com

How reproducible:

always

Steps to Reproduce:
1.Go to openshift and login
2.Create two app as above
3.Visit the url of the two app
  
Actual results:

Two url will redirect to the same page.

Expected results:

Maybe we should not use the upper character for the first letter of every app name.

Additional info:

Comment 1 Rob Millner 2012-06-01 18:03:31 UTC
DNS names are case insensitive.
http://www.ietf.org/rfc/rfc4343.txt

Since the application and domain name directly translate to a DNS name, they should be case insensitive as well; for example by forcing lower case names or testing in a case-insensitive manner.

Comment 2 Clayton Coleman 2012-06-04 00:08:27 UTC
The broker should not be allowing names that differe only by case, not the site.

Comment 3 Vincent Danen 2012-06-06 04:06:51 UTC
Is there a reason this was tagged with the Security keyword?  This does not sound like a security flaw to me.

Comment 4 Abhishek Gupta 2012-06-16 01:59:03 UTC
Fixed and waiting for merge
https://github.com/openshift/crankcase/pull/136

Comment 5 Yujie Zhang 2012-06-19 07:08:49 UTC
(In reply to comment #4)
Tested this issue on devenv_1857, it has been fixed now, thanks.


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