Skip to content
This repository has been archived by the owner on Jan 13, 2023. It is now read-only.

Neighbor URIs must use udp protocol #6

Closed
todofixthis opened this issue Jan 4, 2017 · 2 comments
Closed

Neighbor URIs must use udp protocol #6

todofixthis opened this issue Jan 4, 2017 · 2 comments
Assignees
@todofixthis
Labels
enhancement

Comments

@todofixthis
Copy link
Contributor

The IRI implementation only supports udp:// URIs for neighbors. If you send an addNeighbors API request containing an http:// URI, that URI will be silently ignored.

When validating an addNeighbors or removeNeighbors request, treat any URIs that don't use udp:// protocol as invalid.
@todofixthis todofixthis changed the title Neighbor URIs must use udp protocol. Neighbor URIs must use udp protocol Jan 4, 2017
@todofixthis
Copy link
Contributor Author

Will be delivered with #13

todofixthis added a commit that referenced this issue Jan 26, 2017
@todofixthis
[#6,#13] Cleaned up URI handling.
c3b261d 
- Un-reinvented a wheel.
- HttpAdapter no longer accepts `udp://` URIs.
- `AddNeighborsCommand` and `RemoveNeighborsCommand` now only accepts `udp://` URIs.
@todofixthis
Copy link
Contributor Author

Scheduled for release: 1.1.0

@todofixthis todofixthis self-assigned this Jan 26, 2017
This was referenced Jan 28, 2017
Closed
Merged
marko-k0 pushed a commit to marko-k0/iota.lib.py that referenced this issue Jul 28, 2018
@todofixthis
[iotaledger#6,iotaledger#13] Cleaned up URI handling.
ae3b187 
- Un-reinvented a wheel.
- HttpAdapter no longer accepts `udp://` URIs.
- `AddNeighborsCommand` and `RemoveNeighborsCommand` now only accepts `udp://` URIs.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@todofixthis todofixthis

Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@todofixthis