v1.83

README.md

@@ -13,7 +13,7 @@ HotspotAutoLogin is a script that designed to automate the login process for Wi-
 ### Option 1: Executable Script
 - If you don't want to install Python, you can download the precompiled executable version from the Releases section.
 [Download the Latest Executable (.exe) Release](https://github.com/denizsafak/HotspotAutoLogin/releases/latest)
-- Double-click on HotspotAutoLogin.exe to launch the application. The wallpaper information display will appear on your screen.
+- Double-click on HotspotAutoLogin.exe to launch the application.
 
 ### Option 2: Run with Python
 - Clone or download the repository to your local machine.

main/HotspotAutoLogin.pyw

@@ -16,9 +16,15 @@ dns.resolver.default_resolver = dns.resolver.Resolver()
 from urllib3.exceptions import InsecureRequestWarning
 requests.packages.urllib3.disable_warnings(category=InsecureRequestWarning)
 
+################################# REMOVE THIS BEFORE USING PYINSTALLER #################################
+# Change the current working directory to the script's directory
+script_dir = os.path.dirname(os.path.abspath(__file__))
+os.chdir(script_dir)
+################################# REMOVE THIS BEFORE USING PYINSTALLER #################################
+
 # Program Information
 program_name = "HotspotAutoLogin"
-version = "v1.80"
+version = "v1.83"
 github_link = "https://github.com/denizsafak/HotspotAutoLogin"
 
 # Function to center a window on the screen
@@ -223,6 +229,7 @@ root.mainloop()
 if selected_profile:
     payload = selected_profile.get('payload')
     url = selected_profile.get('url', "")
+    internet_check_url = selected_profile.get('internet_check_url', "")
     ssid = selected_profile.get('ssid', "")
     check_every_second = selected_profile.get('check_every_second', "")
     dialog_geometry = selected_profile.get('dialog_geometry', "")
@@ -249,7 +256,7 @@ def is_internet_available():
         # -n is the number of pings
         # -l is the size of the packet
         # -w is the timeout in seconds
-        subprocess.run(['ping', '8.8.8.8', '-n', '3', '-l', '32', '-w', '20'], check=True, stdout=subprocess.PIPE, stderr=subprocess.PIPE, creationflags=subprocess.CREATE_NO_WINDOW)
+        subprocess.run(['ping', internet_check_url, '-n', '3', '-l', '32', '-w', '20'], check=True, stdout=subprocess.PIPE, stderr=subprocess.PIPE, creationflags=subprocess.CREATE_NO_WINDOW)
         return True
     except subprocess.CalledProcessError:
         return False
@@ -259,7 +266,6 @@ def get_connected_network():
     try:
         startupinfo = subprocess.STARTUPINFO()
         startupinfo.dwFlags |= subprocess.STARTF_USESHOWWINDOW
-        
         # Check if connected to Ethernet
         ethernet_output = subprocess.check_output(["netsh", "interface", "show", "interface"], startupinfo=startupinfo).decode("utf-8")
         ethernet_lines = ethernet_output.split("\n")
@@ -267,7 +273,6 @@ def get_connected_network():
             if "Connected" in line:
                 if "Ethernet" in line:
                     return "Ethernet"
-
         # If not connected to Ethernet, check Wi-Fi SSID
         wifi_output = subprocess.check_output(["netsh", "wlan", "show", "interfaces"], startupinfo=startupinfo).decode("utf-8")
         wifi_lines = wifi_output.split("\n")
@@ -283,6 +288,21 @@ def get_connected_network():
     except subprocess.CalledProcessError:
         return False
     
+# Is connected to wifi
+def is_connected_to_wifi():
+    try:
+        startupinfo = subprocess.STARTUPINFO()
+        startupinfo.dwFlags |= subprocess.STARTF_USESHOWWINDOW
+        wifi_output = subprocess.check_output(["netsh", "wlan", "show", "interfaces"], startupinfo=startupinfo).decode("utf-8")
+        wifi_lines = wifi_output.split("\n")
+        for line in wifi_lines:
+            if "SSID" in line:
+                return True
+        else:
+            return False
+    except subprocess.CalledProcessError:
+        return False
+    
 # Function to exit the application
 def exit_application(icon, item):
     icon.stop()
@@ -416,6 +436,13 @@ def check_network_status():
         if (connected_ssid):
             connected_ssid_lower = connected_ssid.lower()
         if (connected_ssid) and ((connected_ssid_lower == expected_ssid_lower)):
+            if is_connected_to_wifi() and (connected_ssid == "Ethernet" and expected_ssid == "Ethernet"):
+                message = "You are both connected to Wi-Fi and Ethernet. Disconnecting from Wi-Fi and continuing with Ethernet..."
+                add_to_log(message)
+                startupinfo = subprocess.STARTUPINFO()
+                startupinfo.dwFlags |= subprocess.STARTF_USESHOWWINDOW
+                subprocess.check_output(['netsh', 'wlan', 'disconnect', 'interface=' + "Wi-Fi"], startupinfo=startupinfo).decode("utf-8")
+                time.sleep(5)
             if (is_internet_available()):
                 sleepcount = check_every_second
                 request_success = False
@@ -450,7 +477,7 @@ def check_network_status():
                         save_to_file(message)
                 except requests.exceptions.RequestException as e:
                     errorcount += 1
-                    sleepcount = 1
+                    sleepcount = 3
                     request_success = False
                     if isinstance(e, requests.exceptions.HTTPError):
                         # Handle HTTP errors with different messages for different status codes
@@ -466,13 +493,11 @@ def check_network_status():
                         else:
                             message = "Request failed with HTTP error ({}). Trying to reconnect to the network... (Errors: {}/10)".format(status_code, errorcount)
                     else:
-                        message = "Request failed with error:{}. If you are usşng a VPN, please disable it for now. Sometimes 'Kill Switch' feature cause this error. Trying to reconnect to the network... (Errors: {}/10)".format(e, errorcount)
+                        message = "Request failed with error: {}. If you are usşng a VPN, please disable it for now. Trying to reconnect to the network... (Errors: {}/10)".format(e, errorcount)
                     add_to_log(message)
                     save_to_file(message)
                     time.sleep(sleepcount)
                     # If the payload URL is not reachable, try to reconnect to the network
-                    startupinfo = subprocess.STARTUPINFO()
-                    startupinfo.dwFlags |= subprocess.STARTF_USESHOWWINDOW
                     if connected_ssid == "Ethernet":
                         try:
                             # Disable and enable the Ethernet adapter
@@ -494,6 +519,8 @@ def check_network_status():
                             # Disconnect and reconnect to the Wi-Fi network
                             message = "Disconnecting from {}...".format(ssid)
                             add_to_log(message)
+                            startupinfo = subprocess.STARTUPINFO()
+                            startupinfo.dwFlags |= subprocess.STARTF_USESHOWWINDOW
                             subprocess.check_output(['netsh', 'wlan', 'disconnect', 'interface=' + "Wi-Fi"], startupinfo=startupinfo).decode("utf-8")
                             time.sleep(5)
                             subprocess.check_output(['netsh', 'wlan', 'connect', 'name=' + ssid], startupinfo=startupinfo).decode("utf-8")
@@ -518,10 +545,15 @@ def check_network_status():
                         wifi_enabled = True
                         break
             if connected_ssid == "Ethernet":
-                sleepcount = 60
+                sleepcount = 30
                 message = "If you want to connect to {}, please unplug your ethernet cable or disconnect from {}. Checking again in {} seconds...".format(expected_ssid, connected_ssid, str(sleepcount))
                 errorcount = 0
                 add_to_log(message)
+            elif expected_ssid == "Ethernet":
+                sleepcount = 30
+                message = "Looks like your ethernet cable is not plugged in. Please plug in your ethernet cable. Checking again in {} seconds...".format(str(sleepcount))
+                errorcount = 0
+                add_to_log(message)
             elif wifi_enabled:
                 if (connected_ssid):
                     sleepcount = 10
@@ -535,12 +567,14 @@ def check_network_status():
                     result = subprocess.run(['netsh', 'wlan', 'connect', 'name=' + expected_ssid], capture_output=True, text=True, startupinfo=startupinfo)
                     if result.returncode != 0:
                         if result.returncode == 1:
-                            sleepcount = 60
-                            raise ValueError("Could not find a Wi-Fi network with SSID: {}. Checking again in {} seconds...".format(expected_ssid, str(sleepcount)))
+                            sleepcount = 20
+                            message = "Could not find a Wi-Fi network with SSID: {}. If your Wi-Fi is disabled, please enable it. Checking again in {} seconds...".format(expected_ssid, str(sleepcount))
+                            add_to_log(message)
                         else:
                             raise subprocess.CalledProcessError(result.returncode, result.args, result.stderr)
-                    message = "Connected to {}. Running the script again in {} seconds...".format(expected_ssid, str(sleepcount))
-                    add_to_log(message)
+                    else:
+                        message = "Connected to {}. Running the script again in {} seconds...".format(expected_ssid, str(sleepcount))
+                        add_to_log(message)
                 except (subprocess.CalledProcessError, ValueError) as e:
                     errorcount += 1
                     sleepcount = 15

main/config.json

@@ -4,6 +4,7 @@
             "name": "NAME_OF_YOUR_PROFILE (WI-FI EXAMPLE)",
             "ssid": "SSID_OF_YOUR_WIFI",
             "url": "PAYLOAD_URL_HERE",
+            "internet_check_url": "8.8.8.8",
             "payload": {
                 "username": "PLEASE WRITE YOUR OWN PAYLOAD VALUES HERE. THIS IS AN EXAMPLE",
                 "password": "PLEASE WRITE YOUR OWN PAYLOAD VALUES HERE. THIS IS AN EXAMPLE"
@@ -22,6 +23,7 @@
         {
             "name": "NAME_OF_YOUR_PROFILE (ETHERNET EXAMPLE)",
             "url": "PAYLOAD_URL_HERE",
+            "internet_check_url": "8.8.8.8",
             "payload": "&auth_user=XXXX%40gmail.com&auth_pass=xxx&redirurl=&accept=Login",
             "headers": {
                 "Content-Type": "application/x-www-form-urlencoded",

main/refrenv.bat

@@ -0,0 +1,359 @@
+<!-- : Begin batch script
+@echo off
+REM PUSHD "%~dp0"
+
+
+REM author: Badr Elmers 2021-2023
+REM description: refrenv = refresh environment. this is a better alternative to the chocolatey refreshenv for cmd
+REM version: 1.0
+REM https://github.com/badrelmers/RefrEnv
+REM https://stackoverflow.com/questions/171588/is-there-a-command-to-refresh-environment-variables-from-the-command-prompt-in-w
+
+REM ___USAGE_____________________________________________________________
+REM usage: 
+REM        call refrenv.bat        full refresh. refresh all non critical variables*, and refresh the PATH
+
+REM debug:
+REM        to debug what this script do create this variable in your parent script like that
+REM        set debugme=yes
+REM        then the folder containing the files used to set the variables will be open. Then see
+REM        _NewEnv.cmd this is the file which run inside your script to setup the new variables, you
+REM        can also revise the intermediate files _NewEnv.cmd_temp_.cmd and _NewEnv.cmd_temp2_.cmd 
+REM        (those two contains all the variables before removing the duplicates and the unwanted variables)
+
+
+REM you can also put this script in windows\systems32 or another place in your %PATH% then call it from an interactive console by writing refrenv
+
+REM *critical variables: are variables which belong to cmd/windows and should not be refreshed normally like:
+REM - windows vars:
+REM ALLUSERSPROFILE APPDATA CommonProgramFiles CommonProgramFiles(x86) CommonProgramW6432 COMPUTERNAME ComSpec HOMEDRIVE HOMEPATH LOCALAPPDATA LOGONSERVER NUMBER_OF_PROCESSORS OS PATHEXT PROCESSOR_ARCHITECTURE PROCESSOR_ARCHITEW6432 PROCESSOR_IDENTIFIER PROCESSOR_LEVEL PROCESSOR_REVISION ProgramData ProgramFiles ProgramFiles(x86) ProgramW6432 PUBLIC SystemDrive SystemRoot TEMP TMP USERDOMAIN USERDOMAIN_ROAMINGPROFILE USERNAME USERPROFILE windir SESSIONNAME
+
+
+REM ___INFO_____________________________________________________________
+REM :: this script reload environment variables inside cmd every time you want environment changes to propagate, so you do not need to restart cmd after setting a new variable with setx or when installing new apps which add new variables ...etc
+
+
+REM This is a better alternative to the chocolatey refreshenv for cmd, which solves a lot of problems like:
+
+REM The Chocolatey refreshenv is so bad if the variable have some cmd meta-characters, see this test:
+    REM add this to the path in HKCU\Environment: test & echo baaaaaaaaaad, and run the chocolatey refreshenv you will see that it prints baaaaaaaaaad which is very bad, and the new path is not added to your path variable.
+    REM This script solve this and you can test it with any meta-character, even something so bad like:
+    REM ; & % ' ( ) ~ + @ # $ { } [ ] , ` ! ^ | > < \ / " : ? * = . - _ & echo baaaad
+    
+REM refreshenv adds only system and user environment variables, but CMD adds volatile variables too (HKCU\Volatile Environment). This script will merge all the three and remove any duplicates.
+
+REM refreshenv reset your PATH. This script append the new path to the old path of the parent script which called this script. It is better than overwriting the old path, otherwise it will delete any newly added path by the parent script.
+
+REM This script solve this problem described in a comment by @Gene Mayevsky: refreshenv modifies env variables TEMP and TMP replacing them with values stored in HKCU\Environment. In my case I run the script to update env variables modified by Jenkins job on a slave that's running under SYSTEM account, so TEMP and TMP get substituted by %USERPROFILE%\AppData\Local\Temp instead of C:\Windows\Temp. This breaks build because linker cannot open system profile's Temp folder.
+
+REM ________
+REM this script solve things like that too:
+REM The confusing thing might be that there are a few places to start the cmd from. In my case I run cmd from windows explorer and the environment variables did not change while when starting cmd from the "run" (windows key + r) the environment variables were changed.
+
+REM In my case I just had to kill the windows explorer process from the task manager and then restart it again from the task manager.
+REM Once I did this I had access to the new environment variable from a cmd that was spawned from windows explorer.
+
+REM my conclusion:
+REM if I add a new variable with setx, i can access it in cmd only if i run cmd as admin, without admin right i have to restart explorer to see that new variable. but running this script inside my script (who sets the variable with setx) solve this problem and i do not have to restart explorer
+
+
+REM ________
+REM windows recreate the path using three places at less:
+REM the User namespace:    HKCU\Environment
+REM the System namespace:  HKLM\SYSTEM\CurrentControlSet\Control\Session Manager\Environment
+REM the Session namespace: HKCU\Volatile Environment
+REM but the original chocolatey script did not add the volatile path. This script will merge all the three and remove any duplicates. this is what windows do by default too
+
+REM there is this too which cmd seems to read when first running, but it contains only TEMP and TMP,so i will not use it
+REM HKEY_USERS\.DEFAULT\Environment
+
+
+REM ___TESTING_____________________________________________________________
+REM to test this script with extreme cases do
+    REM :: Set a bad variable
+    REM add a var in reg HKCU\Environment as the following, and see that echo is not executed.  if you use refreshenv of chocolatey you will see that echo is executed which is so bad!
+    REM so save this in reg:
+    REM all 32 characters: & % ' ( ) ~ + @ # $ { } [ ] ; , ` ! ^ | > < \ / " : ? * = . - _ & echo baaaad
+    REM and this:
+    REM (^.*)(Form Product=")([^"]*") FormType="[^"]*" FormID="([0-9][0-9]*)".*$
+    REM and use set to print those variables and see if they are saved without change ; refreshenv fail dramatically with those variables
+    
+    
+REM invalid characters (illegal characters in file names) in Windows using NTFS
+REM \ / : * ? "  < > |  and ^ in FAT 
+
+
+
+REM __________________________________________________________________________________________
+REM __________________________________________________________________________________________
+REM __________________________________________________________________________________________
+REM this is a hybrid script which call vbs from cmd directly
+REM :: The only restriction is the batch code cannot contain - - > (without space between - - > of course)
+REM :: The only restriction is the VBS code cannot contain </script>.
+REM :: The only risk is the undocumented use of "%~f0?.wsf" as the script to load. Somehow the parser properly finds and loads the running .BAT script "%~f0", and the ?.wsf suffix mysteriously instructs CSCRIPT to interpret the script as WSF. Hopefully MicroSoft will never disable that "feature".
+REM :: https://stackoverflow.com/questions/9074476/is-it-possible-to-embed-and-execute-vbscript-within-a-batch-file-without-using-a
+
+if "%debugme%"=="yes" (
+    echo RefrEnv - Refresh the Environment for CMD - ^(Debug enabled^)
+) else (
+    echo RefrEnv - Refresh the Environment for CMD
+)
+
+set "TEMPDir=%TEMP%\refrenv"
+IF NOT EXIST "%TEMPDir%" mkdir "%TEMPDir%"
+set "outputfile=%TEMPDir%\_NewEnv.cmd"
+
+
+REM detect if DelayedExpansion is enabled
+REM It relies on the fact, that the last caret will be removed only in delayed mode.
+REM https://www.dostips.com/forum/viewtopic.php?t=6496
+set "DelayedExpansionState=IsDisabled"
+IF "^!" == "^!^" (
+    REM echo DelayedExpansion is enabled
+    set "DelayedExpansionState=IsEnabled"
+)
+
+
+REM :: generate %outputfile% which contain all the new variables
+REM cscript //nologo "%~f0?.wsf" %1
+cscript //nologo "%~f0?.wsf" "%outputfile%" %DelayedExpansionState%
+
+
+REM ::set the new variables generated with vbscript script above
+REM for this to work always it is necessary to use DisableDelayedExpansion or escape ! and ^ when using EnableDelayedExpansion, but this script already solve this, so no worry about that now, thanks to God
+REM test it with some bad var like:
+REM all 32 characters: ; & % ' ( ) ~ + @ # $ { } [ ] , ` ! ^ | > < \ / " : ? * = . - _ & echo baaaad
+REM For /f delims^=^ eol^= %%a in (%outputfile%) do %%a
+REM for /f "delims== tokens=1,2" %%G in (%outputfile%) do set "%%G=%%H"
+For /f delims^=^ eol^= %%a in (%outputfile%) do set %%a
+
+
+REM for safely print a variable with bad charachters do:
+REM SETLOCAL EnableDelayedExpansion
+REM echo "!z9!"
+REM or
+REM set z9
+REM but generally paths and environment variables should not have bad metacharacters, but it is not a rule!
+
+
+if "%debugme%"=="yes" (
+    explorer "%TEMPDir%"
+) else (
+    rmdir /Q /S "%TEMPDir%"
+)
+
+REM cleanup
+set "TEMPDir="
+set "outputfile="
+set "DelayedExpansionState="
+set "debugme="
+
+
+REM pause
+exit /b
+
+
+
+REM #############################################################################
+REM :: to run jscript you have to put <script language="JScript"> directly after ----- Begin wsf script --->
+----- Begin wsf script --->
+<job><script language="VBScript">
+REM #############################################################################
+REM ### put you code here #######################################################
+REM #############################################################################
+
+REM based on itsadok script from here
+REM https://stackoverflow.com/questions/171588/is-there-a-command-to-refresh-environment-variables-from-the-command-prompt-in-w
+
+REM and it is faster as stated by this comment
+REM While I prefer the Chocolatey code-wise for being pure batch code, overall I decided to use this one, since it's faster. (~0.3 seconds instead of ~1 second -- which is nice, since I use it frequently in my Explorer "start cmd here" entry) – 
+
+REM and it is safer based on my tests, the Chocolatey refreshenv is so bad if the variable have some cmd metacharacters
+
+
+Const ForReading = 1 
+Const ForWriting = 2
+Const ForAppending = 8 
+
+Set WshShell = WScript.CreateObject("WScript.Shell")
+filename=WScript.Arguments.Item(0)
+DelayedExpansionState=WScript.Arguments.Item(1)
+
+TMPfilename=filename & "_temp_.cmd"
+Set fso = CreateObject("Scripting.fileSystemObject")
+Set tmpF = fso.CreateTextFile(TMPfilename, TRUE)
+
+
+set oEnvS=WshShell.Environment("System")
+for each sitem in oEnvS
+    tmpF.WriteLine(sitem)
+next
+SystemPath = oEnvS("PATH")
+
+set oEnvU=WshShell.Environment("User")
+for each sitem in oEnvU
+    tmpF.WriteLine(sitem)
+next
+UserPath = oEnvU("PATH")
+
+set oEnvV=WshShell.Environment("Volatile")
+for each sitem in oEnvV
+    tmpF.WriteLine(sitem)
+next
+VolatilePath = oEnvV("PATH")
+
+set oEnvP=WshShell.Environment("Process")
+REM i will not save the process env but only its path, because it have strange variables like  =::=::\ and  =F:=.... which seems to be added by vbscript
+REM for each sitem in oEnvP
+    REM tmpF.WriteLine(sitem)
+REM next
+REM here we add the actual session path, so we do not reset the original path, because maybe the parent script added some folders to the path, If we need to reset the path then comment the following line
+ProcessPath = oEnvP("PATH")
+
+REM merge System, User, Volatile, and process PATHs
+NewPath = SystemPath & ";" & UserPath & ";" & VolatilePath & ";" & ProcessPath
+
+
+REM ________________________________________________________________
+REM :: remove duplicates from path
+REM :: expand variables so they become like windows do when he read reg and create path, then Remove duplicates without sorting 
+    REM why i will clean the path from duplicates? because:
+    REM the maximum string length in cmd is 8191 characters. But string length doesnt mean that you can save 8191 characters in a variable because also the assignment belongs to the string. you can save 8189 characters because the remaining 2 characters are needed for "a="
+   
+    REM based on my tests: 
+    REM when i open cmd as user , windows does not remove any duplicates from the path, and merge system+user+volatil path
+    REM when i open cmd as admin, windows do: system+user path (here windows do not remove duplicates which is stupid!) , then it adds volatil path after removing from it any duplicates 
+
+REM ' https://www.rosettacode.org/wiki/Remove_duplicate_elements#VBScript
+Function remove_duplicates(list)
+	arr = Split(list,";")
+	Set dict = CreateObject("Scripting.Dictionary")
+    REM ' force dictionary compare to be case-insensitive , uncomment to force case-sensitive
+    dict.CompareMode = 1
+
+	For i = 0 To UBound(arr)
+		If dict.Exists(arr(i)) = False Then
+			dict.Add arr(i),""
+		End If
+	Next
+	For Each key In dict.Keys
+		tmp = tmp & key & ";"
+	Next
+	remove_duplicates = Left(tmp,Len(tmp)-1)
+End Function
+ 
+REM expand variables
+NewPath = WshShell.ExpandEnvironmentStrings(NewPath)
+REM remove duplicates
+NewPath=remove_duplicates(NewPath)
+
+REM remove_duplicates() will add a ; to the end so lets remove it if the last letter is ;
+If Right(NewPath, 1) = ";" Then 
+    NewPath = Left(NewPath, Len(NewPath) - 1) 
+End If
+  
+tmpF.WriteLine("PATH=" & NewPath)
+tmpF.Close
+
+REM ________________________________________________________________
+REM :: exclude setting variables which may be dangerous to change
+
+    REM when i run a script from task scheduler using SYSTEM user the following variables are the differences between the scheduler env and a normal cmd script, so i will not override those variables
+    REM APPDATA=D:\Users\LLED2\AppData\Roaming
+    REM APPDATA=D:\Windows\system32\config\systemprofile\AppData\Roaming
+
+    REM LOCALAPPDATA=D:\Users\LLED2\AppData\Local
+    REM LOCALAPPDATA=D:\Windows\system32\config\systemprofile\AppData\Local
+
+    REM TEMP=D:\Users\LLED2\AppData\Local\Temp
+    REM TEMP=D:\Windows\TEMP
+
+    REM TMP=D:\Users\LLED2\AppData\Local\Temp
+    REM TMP=D:\Windows\TEMP
+
+    REM USERDOMAIN=LLED2-PC
+    REM USERDOMAIN=WORKGROUP
+
+    REM USERNAME=LLED2
+    REM USERNAME=LLED2-PC$
+
+    REM USERPROFILE=D:\Users\LLED2
+    REM USERPROFILE=D:\Windows\system32\config\systemprofile
+
+    REM i know this thanks to this comment
+    REM The solution is good but it modifies env variables TEMP and TMP replacing them with values stored in HKCU\Environment. In my case I run the script to update env variables modified by Jenkins job on a slave that's running under SYSTEM account, so TEMP and TMP get substituted by %USERPROFILE%\AppData\Local\Temp instead of C:\Windows\Temp. This breaks build because linker cannot open system profile's Temp folder. – Gene Mayevsky Sep 26 '19 at 20:51
+
+
+REM Delete Lines of a Text File Beginning with a Specified String
+REM those are the variables which should not be changed by this script 
+arrBlackList = Array("ALLUSERSPROFILE=", "APPDATA=", "CommonProgramFiles=", "CommonProgramFiles(x86)=", "CommonProgramW6432=", "COMPUTERNAME=", "ComSpec=", "HOMEDRIVE=", "HOMEPATH=", "LOCALAPPDATA=", "LOGONSERVER=", "NUMBER_OF_PROCESSORS=", "OS=", "PATHEXT=", "PROCESSOR_ARCHITECTURE=", "PROCESSOR_ARCHITEW6432=", "PROCESSOR_IDENTIFIER=", "PROCESSOR_LEVEL=", "PROCESSOR_REVISION=", "ProgramData=", "ProgramFiles=", "ProgramFiles(x86)=", "ProgramW6432=", "PUBLIC=", "SystemDrive=", "SystemRoot=", "TEMP=", "TMP=", "USERDOMAIN=", "USERDOMAIN_ROAMINGPROFILE=", "USERNAME=", "USERPROFILE=", "windir=", "SESSIONNAME=")
+
+Set objFS = CreateObject("Scripting.FileSystemObject")
+Set objTS = objFS.OpenTextFile(TMPfilename, ForReading)
+strContents = objTS.ReadAll
+objTS.Close
+
+TMPfilename2= filename & "_temp2_.cmd"
+arrLines = Split(strContents, vbNewLine)
+Set objTS = objFS.OpenTextFile(TMPfilename2, ForWriting, True)
+
+REM this is the equivalent of findstr /V /I /L  or  grep -i -v  , i don t know a better way to do it, but it works fine
+For Each strLine In arrLines
+    bypassThisLine=False
+    For Each BlackWord In arrBlackList
+        If Left(UCase(LTrim(strLine)),Len(BlackWord)) = UCase(BlackWord) Then
+            bypassThisLine=True
+        End If
+    Next
+    If bypassThisLine=False Then
+        objTS.WriteLine strLine
+    End If
+Next
+
+REM ____________________________________________________________
+REM :: expand variables because registry save some variables as unexpanded %....%
+REM :: and escape ! and ^ for cmd EnableDelayedExpansion mode
+
+set f=fso.OpenTextFile(TMPfilename2,ForReading)
+REM Write file:  ForAppending = 8 ForReading = 1 ForWriting = 2 , True=create file if not exist
+set fW=fso.OpenTextFile(filename,ForWriting,True)
+Do Until f.AtEndOfStream
+    LineContent = f.ReadLine
+    REM expand variables
+    LineContent = WshShell.ExpandEnvironmentStrings(LineContent)
+    
+    REM _____this part is so important_____
+    REM if cmd delayedexpansion is enabled in the parent script which calls this script then bad thing happen to variables saved in the registry if they contain ! . if var have ! then ! and ^ are removed; if var do not have ! then ^ is not removed . to understand what happens read this :
+    REM how cmd delayed expansion parse things
+    REM https://stackoverflow.com/questions/4094699/how-does-the-windows-command-interpreter-cmd-exe-parse-scripts/7970912#7970912
+    REM For each parsed token, first check if it contains any !. If not, then the token is not parsed - important for ^ characters. If the token does contain !, then scan each character from left to right:
+    REM     - If it is a caret (^) the next character has no special meaning, the caret itself is removed
+    REM     - If it is an exclamation mark, search for the next exclamation mark (carets are not observed anymore), expand to the value of the variable.
+    REM         - Consecutive opening ! are collapsed into a single !
+    REM         - Any remaining unpaired ! is removed
+    REM ...
+    REM Look at next string of characters, breaking before !, :, or <LF>, and call them VAR
+
+    REM conclusion:
+    REM when delayedexpansion is enabled and var have ! then i have to escape ^ and ! ,BUT IF VAR DO NOT HAVE ! THEN DO NOT ESCAPE ^  .this made me crazy to discover
+    REM when delayedexpansion is disabled then i do not have to escape anything
+    
+    If DelayedExpansionState="IsEnabled" Then
+        If InStr(LineContent, "!") > 0 Then
+            LineContent=Replace(LineContent,"^","^^")
+            LineContent=Replace(LineContent,"!","^!")
+        End If
+    End If
+    REM __________
+    
+    fW.WriteLine(LineContent)
+Loop
+
+f.Close
+fW.Close
+
+REM #############################################################################
+REM ### end of vbscript code ####################################################
+REM #############################################################################
+REM this must be at the end for the hybrid trick, do not remove it
+</script></job>

main/requirements.txt

@@ -0,0 +1,4 @@
+urllib3==1.26.18
+requests==2.32.3
+pystray==0.19.5
+dnspython==2.7.0

requirements.txt

@@ -1,4 +0,0 @@
-urllib3==1.26.18
-requests==2.31.0
-pystray==0.19.5
-dnspython==2.6.1

run.bat

@@ -1,52 +1,118 @@
 @echo off
 setlocal enabledelayedexpansion
+cd /d "%~dp0"
 set NAME=HotspotAutoLogin
-set RUN=HotspotAutoLogin.pyw
-set "requirementsFile=requirements.txt"
+set PROJECTFOLDER=main
+set RUN=%PROJECTFOLDER%\HotspotAutoLogin.pyw
+set requirementsFile=%PROJECTFOLDER%\requirements.txt
 set VENV_PATH=.venv
 set ACTIVATE_PATH=%VENV_PATH%\Scripts\activate
+set CURRENT_DIR=%CD%
+set LAST_DIR_FILE=%PROJECTFOLDER%\last_known_directory.txt
+set refrenv=%PROJECTFOLDER%\refrenv.bat
+set PYTHON_DOWNLOAD_URL=https://www.python.org/ftp/python/3.12.4/python-3.12.4-amd64.exe
+
+:: Display provided argument if any
+if not "%~1"=="" (
+    echo Open with: "%~1"
+)
 
 :: Check if Python is installed
-pip --version >nul 2>&1
-if errorlevel 1 (
-	echo Python is not installed.
-	set /p "userinp=Do you want to install Python (y/n)? "
-	if /i "!userinp!"=="y" (
-		echo Opening download link...
-		start https://www.python.org/downloads/
-		exit /b
+pip --version >nul 2>&1 && (set python_installed=true) || (set python_installed=false)
+if not "%python_installed%"=="true" (
+    echo Python is required to run this program.
+    set /p "userinp=Do you want to install Python (y/n)?: "
+    if /i "!userinp!"=="y" (
+        echo Downloading Python...
+        winget install python --accept-package-agreements
+        if errorlevel 1 (
+            echo Download failed using winget method. Trying to download manually...
+            :bothwingetfailed
+            if not exist "%PROJECTFOLDER%\python_installer.exe" (
+                echo Downloading Python from %PYTHON_DOWNLOAD_URL%
+                curl %PYTHON_DOWNLOAD_URL% -o %PROJECTFOLDER%\python_installer.exe
+                if not exist "%PROJECTFOLDER%\python_installer.exe" (
+                    echo Download failed using curl. Trying with the powershell method
+                    powershell -Command "& {Invoke-WebRequest -Uri '%PYTHON_DOWNLOAD_URL%' -OutFile '%PROJECTFOLDER%\python_installer.exe'}"
+                    if errorlevel 1 (
+                        echo Failed to install Python. Please install Python manually and try again.
+                        pause
+                        exit /b
+                    )
+                )
+            )
+            echo Installing Python. Please wait...
+            start /wait %PROJECTFOLDER%\python_installer.exe /quiet InstallAllUsers=1 PrependPath=1 Include_test=0
+            if errorlevel 1 (
+                echo Failed to install Python. Please install Python manually and try again.
+                pause
+                exit /b
+            )
+            del %PROJECTFOLDER%\python_installer.exe
+            goto python_ok
+        )
     ) else (
-		echo Please install Python and try again.
-		pause
-		exit /b
+        echo Python is required to run this program. Please install Python and try again.
+        pause
+        exit /b
     )
+    :python_ok
+    echo Python installed successfully.
+    :: Refresh the environment variables
+    call %refrenv%
+    set python_installed=true
 )
 
 :: Check if virtual environment exists
 IF NOT EXIST %VENV_PATH% (
-	echo Creating virtual environment...
-	python -m venv %VENV_PATH%
+    echo Creating virtual environment...
+	echo %CURRENT_DIR% > "%LAST_DIR_FILE%"
+    python -m venv %VENV_PATH%
 ) ELSE (
-	goto check_python
+    goto check_python
 )
 
 :: Check if Python exists at the path stored in the virtual environment
-set counut_python=0
+set count_python=0
 :check_python
 %VENV_PATH%\Scripts\python.exe --version >nul 2>&1
 if %errorlevel% neq 0 (
-	set /a count_python+=1
-	echo Python not found at the path stored in the virtual environment.
-	echo Recreating virtual environment...
-	rmdir /s /q %VENV_PATH%
-	python -m venv %VENV_PATH%
-	if !count_python! lss 3 (
-		goto check_python
-	) else (
-		echo Something went wrong. Please try again.
-		pause
-		exit /b
-	)
+    set /a count_python+=1
+    echo Python not found at the path stored in the virtual environment.
+    echo Recreating virtual environment...
+    rmdir /s /q %VENV_PATH%
+    python -m venv %VENV_PATH%
+    if !count_python! lss 3 (
+        goto check_python
+    ) else (
+        echo Something went wrong. Please try again.
+        pause
+        exit /b
+    )
+)
+
+:: Check if the current directory has changed since the last run
+if exist "%LAST_DIR_FILE%" (
+    set /p last_dir=<"%LAST_DIR_FILE%"
+    cd "%CURRENT_DIR%" && (
+        if not exist "!last_dir!" (
+            echo Current directory: %CURRENT_DIR%
+            echo Last known directory: !last_dir!
+            echo Looks like the directory of the program has been changed. 
+            echo It is hihgly recommended to recreate the virtual environment.
+            set /p "recreate_venv=Recreate the virtual environment (y/n)?: "
+            if /i "!recreate_venv!"=="y" (
+                echo Recreating virtual environment...
+                rmdir /s /q %VENV_PATH%
+                python -m venv %VENV_PATH%
+				:: Update the last directory file
+				echo %CURRENT_DIR% > "%LAST_DIR_FILE%"
+            )
+        )
+    )
+) else (
+    :: Create the last directory file if it doesn't exist
+    echo %CURRENT_DIR% > "%LAST_DIR_FILE%"
 )
 
 :: Activate the virtual environment
@@ -55,45 +121,46 @@ set count_activate=0
 echo Activating virtual environment...
 call %ACTIVATE_PATH%
 if errorlevel 1 (
-	set /a count_activate+=1
-	echo Failed to activate virtual environment.
-	echo Recreating virtual environment...
-	rmdir /s /q %VENV_PATH%
-	python -m venv %VENV_PATH%
-	if !count_activate! lss 3 (
-		goto activate
-	) else (
-		echo Failed to activate virtual environment. Please try again.
-		pause
-		exit /b
-	)
+    set /a count_activate+=1
+    echo Failed to activate virtual environment.
+    echo Recreating virtual environment...
+    rmdir /s /q %VENV_PATH%
+    python -m venv %VENV_PATH%
+    if !count_activate! lss 3 (
+        goto activate
+    ) else (
+        echo Failed to activate virtual environment. Please try again.
+        pause
+        exit /b
+    )
 )
 
 :: Get the list of installed packages using pip freeze
 echo Checking the requirements...
-for /f "tokens=1,* delims==" %%i in ('pip freeze') do (
-	set installed[%%i]=%%j
+for /f "tokens=1,* delims==" %%i in ('pip freeze --all') do (
+    set installed[%%i]=%%j
 )
 
 :: Compare with the requirements from the requirements.txt file
 for /f "tokens=1,* delims==" %%i in (%requirementsFile%) do (
-	if not "!installed[%%i]!"=="%%j" (
-		echo Installing package: %%i==%%j
-		pip install %%i==%%j --upgrade --quiet
-		if errorlevel 1 (
-			echo Failed to install %%i==%%j. Please check your internet connection and try again.
-			pause
-			exit /b
-		)
-	)
+    if not "!installed[%%i]!"=="%%j" (
+        echo Installing package: %%i==%%j
+        pip install %%i==%%j --upgrade --quiet
+        if errorlevel 1 (
+            echo Failed to install %%i==%%j. Please check your internet connection and try again.
+            pause
+            exit /b
+        )
+    )
 )
 
 :: Run the program
 echo Starting %NAME%...
-start /B "" "%VENV_PATH%/Scripts/pythonw.exe" %RUN%
+start /B "" "%VENV_PATH%\Scripts\pythonw.exe" %RUN% %* > nul 2>&1
 if errorlevel 1 (
-	echo Failed to start %NAME%. Please try again.
-	pause
-	exit /b
+    echo Failed to start %NAME%. Please try again.
+    pause
+    exit /b
 )
-exit /b
+
+exit /b