Euca2ool.make_connection should use urlparse library
Bug #405628 reported by
mjpan
This bug affects 2 people
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Eucalyptus |
Fix Released
|
Undecided
|
Mitch Garnaat | ||
| euca2ools |
Fix Released
|
Undecided
|
Mitch Garnaat | ||
Bug Description
Applying the attached patch file switches the parsing to using the urlparse library.
I've tried to ensure that the object's attributes all remain the same.
Related branches
Revision history for this message
| mjpan (mjpan) wrote | #1 |
| Changed in eucalyptus: | |
| status: | New → Confirmed |
| status: | Confirmed → Invalid |
| Changed in euca2ools: | |
| status: | New → Confirmed |
Revision history for this message
| Pete Zaitcev (zaitcev) wrote | #2 |
Revision history for this message
| Mitch Garnaat (mitch-garnaat) wrote | #3 |
| Changed in eucalyptus: | |
| assignee: | nobody → Mitch Garnaat (mitch-garnaat) |
| status: | Invalid → Fix Committed |
| Changed in euca2ools: | |
| assignee: | nobody → Mitch Garnaat (mitch-garnaat) |
| Changed in euca2ools: | |
| status: | Confirmed → Fix Committed |
| Changed in euca2ools: | |
| status: | Fix Committed → Fix Released |
| Changed in eucalyptus: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
The existing code is badly broken, no doubt. I am a notice in Python, so I fixed it up
by force, see this:
https:/
However using a library modules seems like a better idea.