Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

User able to increase port number with more than 4 digit. #390

Open
Jacob15 opened this issue Jan 14, 2019 · 1 comment
Open

User able to increase port number with more than 4 digit. #390

Jacob15 opened this issue Jan 14, 2019 · 1 comment
Assignees
@kinfkong
Labels
tcx_FixAccepted

Comments

@Jacob15
Copy link

Jacob15 commented Jan 14, 2019

Steps to Reproduce:

  1. Deploy the app.
  2. Click on Add redis server instance icon.
  3. In the Add redis server dialog increase the port number.
  4. User able to increase port number beyond 4 digit

Actual Result

User able to increase port number beyond 4 digits.

Expected Result

User should not able to increase port number beyond 4 digits.

Frequency
5/5

Device: Laptop
OS: Windows 7 32 bit
Browser: Chrome(latest)

Images

port
@standlove
Copy link
Collaborator

the max value is 65535

kinfkong added a commit to kinfkong/TC-redis-manager that referenced this issue Jan 15, 2019
@kinfkong
fix issue topcoderinc#390
cd7ba50
@kinfkong kinfkong self-assigned this Jan 15, 2019
standlove added a commit that referenced this issue Jan 15, 2019
@standlove
Merge pull request #442 from kinfkong/issue-390
48f1bf6 
Fix issue #390
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kinfkong kinfkong

Labels
tcx_FixAccepted
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@kinfkong @standlove @Jacob15