yvesf / andiodine

Android VPN over DNS (no root required). Repo moved to gitlab
https://gitlab.com/andiodine/andiodine
149 stars 27 forks source link

Setting "Request Type" does not seem to take effect #18

Closed ghost closed 7 years ago

ghost commented 9 years ago

I have set Request Type to something other than AUTODETECT, but it still attempts to autodetect. Is this field being used when connecting?

2m commented 9 years ago

After a quick look at the code it seems that it is not used when calling iodine native code.

ghost commented 9 years ago

Not sure if I'm going crazy, but I think it tried to use a specific record type that I selected, but it wasn't the one I had set at the time.

I also see what you mean about it not being called, so perhaps I am indeed just going crazy.

2m commented 9 years ago

Looking more at the code it seems that it is always autodetecting request type irregardless of the selected request type.

yvesf commented 9 years ago

I can confirm that. Looks like I forgot about that aspect after modeling the Configuration. Basically it's just a matter of converting the Java-Enum to string and adding to Java_org_xapek_andiodine_IodineClient_connect and calling client_set_qtype("A") with it.