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

Pro search does not support IPv6 IP addresses #2550

Closed
matthewjohns0n opened this issue Nov 22, 2022 · 0 comments · Fixed by #2551
Closed

Pro search does not support IPv6 IP addresses #2550

matthewjohns0n opened this issue Nov 22, 2022 · 0 comments · Fixed by #2551
Assignees
@matthewjohns0n
Labels
Bug: Accepted Bug has been confirmed, is reproducible, and ready to work on. EE7
Milestone
7.2.2

Comments

@matthewjohns0n
Copy link
Member

Description of the problem

When using IPv6, pro search does not allow you to search because the database column is too short.

Screenshots / Videos / Template Code

image

Possible Solution

Make the length of the ip_address column longer.
@matthewjohns0n matthewjohns0n added Bug: Accepted Bug has been confirmed, is reproducible, and ready to work on. EE7 labels Nov 22, 2022
@matthewjohns0n matthewjohns0n added this to the 7.2.2 milestone Nov 22, 2022
matthewjohns0n added a commit that referenced this issue Nov 22, 2022
@matthewjohns0n
Lengthen the ip_address column to support IPv6 addresses
710325f 
Fixes #2550
@matthewjohns0n matthewjohns0n mentioned this issue Nov 22, 2022
Merged
matthewjohns0n added a commit that referenced this issue Nov 23, 2022
@matthewjohns0n
Merge pull request #2551 from ExpressionEngine/bug/7.dev/pro_search_i…
9eb1053 
…p_address_ipv6_support

Resolved #2550 where Pro Search did not store IPv6 addresses correctly
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@matthewjohns0n matthewjohns0n

Labels
Bug: Accepted Bug has been confirmed, is reproducible, and ready to work on. EE7
Projects
None yet
Milestone
7.2.2
3 participants
@bryannielsen @intoeetive @matthewjohns0n