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

Not working transformation Ip To Binary #13552

Closed
obeone opened this Issue Aug 2, 2017 · 3 comments

Comments

Projects
None yet
2 participants
@obeone

obeone commented Aug 2, 2017

Steps to reproduce

  1. Set IpToBinary input transformation to a binary column
  2. Set an IP

Expected behaviour

The IP must be convert to binary, and be added in table

Actual behaviour

An error occur, and it won't display the query (on a click on « Edit », an error about malformed UTF-8 comes.

Server configuration

Operating system:
Linux

Web server:
Apache 2.4

Database:
MySQL 5.7

PHP version:
7.0

phpMyAdmin version:
master (up-to-date)

Client configuration

Browser:
Safari

Operating system:
Mac OS 10.13 public beta 3

I'm writing transformation plugins to have input UUID to Binary and output Binary To UUID (when everything works, I will create a merge request).
It's ok for the output, but for input I have this problem, and it's the same than the provided IpToBinary transformation.
As far as I understand, the returned transformation is in binary, but phpMyAdmin want to handle it as string.

@nijel

This comment has been minimized.

Show comment
Hide comment
@nijel

nijel Aug 2, 2017

Member

How much up to date is your master? I've fixed several issues with transformations yesterday, see #13522.

Member

nijel commented Aug 2, 2017

How much up to date is your master? I've fixed several issues with transformations yesterday, see #13522.

@nijel nijel added the bug label Aug 2, 2017

@nijel

This comment has been minimized.

Show comment
Hide comment
@nijel

nijel Aug 2, 2017

Member

Ah, this is about input transformation, then see also #13477

Member

nijel commented Aug 2, 2017

Ah, this is about input transformation, then see also #13477

@obeone

This comment has been minimized.

Show comment
Hide comment
@obeone

obeone Aug 2, 2017

I tried a pull just now, it's really up to date ;)
It's not related with #13477, the input transformation is correctly stored in pma db.

If I try to return a test string, it work. I only have a problem if it is binary data returned, this is why I thought a problem related to that

obeone commented Aug 2, 2017

I tried a pull just now, it's really up to date ;)
It's not related with #13477, the input transformation is correctly stored in pma db.

If I try to return a test string, it work. I only have a problem if it is binary data returned, this is why I thought a problem related to that

@nijel nijel self-assigned this Oct 3, 2017

@nijel nijel added this to the 4.7.5 milestone Oct 3, 2017

@nijel nijel closed this in d0d24ce Oct 3, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment