Jump to content
IGNORED

Firmware 2022-11-05 won't accept my wifi password


Recommended Posts

Firmware version 2022-11-05 shortened the maximum wifi password length: it won't let me type my whole password on initial setup, which prevents it from connecting to my wifi network.

 

I've reverted to the previous version for now, but is this an intended change?

 

Edit: specifically, the configuration program stops me typing after 31 characters. MAX_WIFI_PASS_LEN is 64 in the fujinet-platformio code, but I'm not sure where the code that handles password input on the Atari lives.

Edited by Eyvind Bernhardsen
Details, details
Link to comment
Share on other sites

5 hours ago, Eyvind Bernhardsen said:

Firmware version 2022-11-05 shortened the maximum wifi password length: it won't let me type my whole password on initial setup, which prevents it from connecting to my wifi network.

 

I've reverted to the previous version for now, but is this an intended change?

 

Edit: specifically, the configuration program stops me typing after 31 characters. MAX_WIFI_PASS_LEN is 64 in the fujinet-platformio code, but I'm not sure where the code that handles password input on the Atari lives.

Not an intended feature. Thank you for letting us know, we'll fix it, and do a new build.

 

-Thom

 

  • Like 1
Link to comment
Share on other sites

5 hours ago, Eyvind Bernhardsen said:

...
Edit: specifically, the configuration program stops me typing after 31 characters. MAX_WIFI_PASS_LEN is 64 in the fujinet-platformio code, but I'm not sure where the code that handles password input on the Atari lives.

@Eyvind Bernhardsen If you're interested in what the fix was, it's here: https://github.com/FujiNetWIFI/fujinet-config-adam/commit/a9e08fd28265db37d7f75dec8c3f25b9c32c399d

 

-Thom

  • Thanks 1
Link to comment
Share on other sites

3 hours ago, mistapaul said:

I managed to get my password to work, but in menu the <> don't work nor the highscore save.

 

This particular bug happened because the new CONFIG changed how it mounts devices.

 

In the previous version of CONFIG, each device was individually mounted using the Mount Disk Image command, in a loop, from within CONFIG.

 

Last year, as part of user changes, I added a mount all ATR that you can use instead of the CONFIG program to quickly mount devices and boot. You can select this from the web admin. And as part of this program, I added a new FujiNet command MOUNT ALL, which does this ON the fujinet. This made the mount all ATR very small and simple.

 

This particular MOUNT ALL command was then carried forward into the versions for the Adam, C64, and Apple2.

 

@Schadret had been working for the last few months to add an Atari target to the rewritten CONFIG that is being used currently for Adam, C64, Apple2, adding back in an Atari target, SO THAT ALL TARGETS NOW USE THE EXACT SAME CODE BASE.

 

If you zoom into the firmware and compare the code for mount image, and mount all, they're basically duplicated, and this was my bad. 

https://github.com/FujiNetWIFI/fujinet-platformio/commit/c68d21b5cfa20b2ce412ecec372f27798c8c9485

 

I have applied a fix to the mount all command which properly sets the host variable, so that High Score saving will work, correctly.

 

I have also made a TODO notation to refactor these two commands to use common code. :)

 

With this, @mozzwald will spin a new build shortly. I apologize for any inconveniences caused.

 

-Thom

  • Like 3
  • Thanks 1
Link to comment
Share on other sites

On 11/6/2022 at 1:47 PM, Eyvind Bernhardsen said:

Aha, thanks! That was quick, and now I know where the config program lives :)

So sorry about the bug, an oversight on my part and I didn't think to test with that long a Wifi password!

 

If you notice anything else let us know here or you can enter an issue on github at Issues · FujiNetWIFI/fujinet-config-adam (github.com)

  • Like 2
Link to comment
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.

Guest
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

Loading...
 Share

  • Recently Browsing   0 members

    • No registered users viewing this page.
×
×
  • Create New...