emoacht / SnowyImageCopy

A Windows desktop app to copy images from FlashAir by a wireless connection
https://www.microsoft.com/store/apps/9MTLPNGRW85L
MIT License
32 stars 14 forks source link

Behavior of FlashAir address textbox with IP address #60

Closed vicentegr closed 3 years ago

vicentegr commented 3 years ago

I use TP-LINK MR3020 as a WIFI repeater and I cannot access the card by name, the application does not let me put the ip of the card to locate it. It would be nice to be able to put the ip. Sugerencia de modificación Uso TP-LINK MR3020 como repetidor WIFI y no puedo acceder a la tarjeta por su nombre, la aplicación no me deja poner la ip de la tarjeta para localizarla. Estaría bien poder poner la ip.

emoacht commented 3 years ago

Hello, @vicentegr How long the length of your IP address string is? Do you specify the port number? You should be able to set a string up to 15, the maximum of ordinary IPv4 address, as the root of name field.

vicentegr commented 3 years ago

Thank you very much for your attention. The name field is auto-corrected to "http: // flashair /" automatically, to which I can add the folder structure I want. It is impossible to put an ip address.

emoacht commented 3 years ago

In such case, I would suggest to write the whole path in the notepad and copy & paste it into the name textbox.

vicentegr commented 3 years ago

Ocurrs the same

vicentegr commented 3 years ago

An analysis of the entered text is programmed, which requires a certain format.

emoacht commented 3 years ago

If you don't mind, could you write the path here?

vicentegr commented 3 years ago

192.168.0.1

vicentegr commented 3 years ago

It works, but it has to be written as http://192.168.0.1/

emoacht commented 3 years ago

Okay. it is the usual way to write down a URL path.

dentphoto commented 3 years ago

Hi, emoacht~~

I found the difference in putting the ip address. Are you intended to make this difference?

Snowy 1.9 - : http://192.168.0.140/dcim http://flashair1/dcim
http://flashair/

Snowy 2.0+ : http://192.168.0.140/ http://flashair1/ http://flashair/

Only http://flashair/ can be used in both versions.

vicentegr commented 3 years ago

Congratulations on the application, the operation is correct. Years ago I did not put the "http: //" when writing an ip. I suggest that when the program does not see the "http: //" add it, but do not add "flashair", at least as long as it is an ip. Someone else will the same thing happen to me. And thank you very much for your attention.

vicentegr commented 3 years ago

"http://192.168.0.1/dcim" used in "Snowy + 2.2.4.0"

emoacht commented 3 years ago

There should be no such difference. Ver 2.* can accept the path after the root as well. I have no plan to introduce a special treatment for IP address because I think it is relatively rare case and checking whether a path is IP address or not would require a bunch of code.

dentphoto commented 3 years ago

That's weird.

In Snowy 1.9 and earlier, the address below does not work at all. http://192.168.0.140/ http://flashair1/ I must add /dcim for flashair recognition.

On the other hand, in Snowy Plus, it works and transfers without /dcim.

emoacht commented 3 years ago

@vicentegr, @dentphoto, please don't mix up topics. I moved some comments to #63.

emoacht commented 3 years ago

@dentphoto There should be no such difference. The path after the root address was implemented 5 years back by https://github.com/emoacht/SnowyImageCopy/commit/0add949f5c4d6a0412c4d7ea1d348766ec0b3edc I think it was likely caused by any other factor.