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.

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

listPublicIPAddresses return not filtered for current network #3295

Closed
DagSonsteboSB opened this issue Apr 25, 2019 · 1 comment · Fixed by #3368
Closed

listPublicIPAddresses return not filtered for current network #3295

DagSonsteboSB opened this issue Apr 25, 2019 · 1 comment · Fixed by #3368
Milestone

Comments

@DagSonsteboSB
Copy link

DagSonsteboSB commented Apr 25, 2019

ISSUE TYPE
  • Bug Report
COMPONENT NAME
  • CloudStack GUI
CLOUDSTACK VERSION
  • ACS 4.11.2
CONFIGURATION
  • Advanced zone
  • 2 x shared networks in "setup" state
OS / ENVIRONMENT
  • N/A
SUMMARY
  • GUI calls listPublicIPAddresses as follows and does not filter for current network when using the "change IP address" function:
    command=listPublicIpAddresses&response=json&allocatedonly=false&networkid=01891a96-1a86-4401-948a-ed434c5be1ba&forvirtualnetwork=false
  • As a result all IP addresses from both shared networks are returned, allowing the end user to select an IP address which can not be assigned to the NIC in question
  • Checking http://cloudstack.apache.org/api/apidocs-4.11/apis/listPublicIpAddresses.html it seems the parameter "networkid" is not actually implemented, however this plays out as follows:
    ** Documentation specifies this as parameter "associatednetworkid"
    ** CloudMonkey etc. will return "associatednetworkid" when querying input parameters
    ** However the API will not actually filter for "networkid" and using "associatednetworkid" returns 0 nil records, hence matching does not work
STEPS TO REPRODUCE
  • Create two shared networks and make sure they are in "setup" state
  • Create VM with NIC assigned to shared net 1
  • Try to use the "change IP" function in the GUI
  • Observe this returns a list of ALL shared IP addresses for both shared networks
  • For completeness sake try to assign an IP address on the second shared network and observe this failing whether VM is running or not
EXPECTED RESULTS
  • GUI should return only IP addresses on current shared network
ACTUAL RESULTS
  • GUI returns list of IP addresses from both shared networks
@DagSonsteboSB DagSonsteboSB changed the title GUI listPublicIPAddresses return not filtered for current network listPublicIPAddresses return not filtered for current network Apr 25, 2019
@rohityadavcloud rohityadavcloud added this to the 4.13.0.0 milestone May 27, 2019
@rohityadavcloud
Copy link
Member

Fixed in #3368

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants