Skip to content

Migliorata compatibilità iOS#24

Closed
mauromarzocca wants to merge 6 commits intofexh10:iosCompatibilityfrom
mauromarzocca:iosCompatibility
Closed

Migliorata compatibilità iOS#24
mauromarzocca wants to merge 6 commits intofexh10:iosCompatibilityfrom
mauromarzocca:iosCompatibility

Conversation

@mauromarzocca
Copy link

Ho migliorato la compatibilità iOS, i test sono stati effettuati su iSH Beta (proveniente da TestFlight) ed iOS 26.0.1.

Ecco una serie di cambiamenti:

  1. Correzioni di Connettività e Sicurezza
  • Aggiornamento dell'URL del Sito: Il primo problema di connessione era dovuto a un indirizzo web obsoleto. Ho aggiornato l'URL da animeworld.so a www.animeworld.ac per puntare al sito corretto.

  • Miglioramento della Gestione della Sessione: Per evitare che le richieste venissero bloccate, ho sostituito le singole chiamate di rete con un oggetto requests.Session. Questo permette di gestire cookie e sessioni in modo più simile a un browser, migliorando l'affidabilità.

  • Risoluzione dell'Errore SSL (Soluzione Finale): Il problema più ostico è stato l'errore SSLCertVerificationError, causato dall'ambiente (probabilmente da iSH, seguiranno ulteriori test) che non riusciva a verificare il certificato di sicurezza del sito. Dopo aver provato diverse soluzioni sicure senza successo, ho provveduto a disabilitare la verifica SSL. Ho aggiunto un avviso di sicurezza in giallo che appare all'inizio per informarti dei rischi associati.

  1. Miglioramenti per la Compatibilità con iOS
  • Rilevamento Corretto di iOS: L'applicazione ora è in grado di riconoscere quando viene eseguita su iOS, permettendo di attivare logiche specifiche per il tuo dispositivo.

  • Nessuna Richiesta di Configurazione Player: Grazie al rilevamento di iOS, l'applicazione non ti chiederà più di configurare manualmente il percorso di VLC.

  • Risoluzione della Chiusura Improvvisa di VLC: Ho corretto il problema per cui VLC si chiudeva subito dopo l'apertura. Ora lo script si mette in pausa e attende che tu prema "Invio" prima di continuare, dando tutto il tempo necessario per guardare l'episodio.

  1. Miglioramenti della Stabilità e Qualità del Codice
  • Correzione dell'Errore di Sintassi Iniziale: Il primissimo problema riscontrato era un SyntaxError che causava il crash dell'applicazione.

  • Eliminazione Completa dei ResourceWarning: ci sono stati degli avvisi relativi a "file non chiusi" e "processi ancora in esecuzione". Questi erano causati da un metodo di programmazione obsoleto (os.popen). Ho modernizzato tutto il codice, sostituendo ogni singola chiamata a os.popen con il più sicuro e robusto subprocess.run. Questo ha eliminato completamente quegli avvisi e ha reso l'applicazione più stabile e affidabile.

  • Risoluzione del Blocco dell'Applicazione: La modernizzazione del codice aveva introdotto un bug che bloccava l'app sulla schermata di configurazione. Ho corretto anche questo, assicurandomi che i menu interattivi ora funzionino correttamente.

In sintesi, tramite un percorso di debug, ho provveduto a risolvere una serie di problemi riscontrati, rendendo l'applicazione pienamente funzionante in questo ambiente specifico e migliorandone la qualità generale.
Non escludo la possibilità che ci siano state migliorie generali, ma provvederò a testarle successivamente, ecco una serie di test che verranno eseguiti:

  • Ubuntu Desktop
  • Windows 11
  • MacOS
  • Android
  • iOS (con TestFlight non beta)

@mauromarzocca mauromarzocca mentioned this pull request Oct 10, 2025
@axtrat
Copy link
Collaborator

axtrat commented Oct 10, 2025

In sintesi, tramite un percorso di debug, ho provveduto a risolvere una serie di problemi riscontrati, rendendo l'applicazione pienamente funzionante in questo ambiente specifico e migliorandone la qualità generale.
Non escludo la possibilità che ci siano state migliorie generali, ma provvederò a testarle successivamente, ecco una serie di test che verranno eseguiti:

Inanzi tutto ti ringrazio per la tua disponibilità nel voler mantenere funzionante la compatibilità IOS.
Però ti consiglierei di non modificare ulteriormente il codice principale, non perché non vogliamo ma, poiché si basa su una versione datata del codice che potrebbe non essere compatibile con quella che stiamo sviluppando.

Inoltre, visto il tuo impegno volevo renderti partecipe della discussione riguardo il futuro della compatibilità IOS #25

@mauromarzocca
Copy link
Author

mauromarzocca commented Oct 10, 2025

Ho preferito modificare solamente il branch relativo ad iOS, ho sistemato la compatibilità con altri sistemi in quanto erano problematiche emerse durante il debug, quindi ho preferito risolverle (per il futuro non si può mai sapere).
Con l'ultima PR, ho fixato tutte le problematiche emerse con iSH, almeno per il momento.
Ho migliorato la documentazione, lasciandola pressoché inalterata, ma migliorando la lettura in markdown.
Il problema principale risiede sopratutto nel certificato SSL, è stato inserito un messaggio di warning, ma, in termini di sicurezza, non si può mai sapere quali problematiche potrebbero emergere in futuro.

Dai miei test, questo script risulta funzionante sui seguenti sistemi:

  • MacOS
  • Ubuntu
  • Windows (via WSL)
  • Android
  • iOS

Questa è l'ultima PR relativa a questa problematiche. #26

@axtrat
Copy link
Collaborator

axtrat commented Oct 11, 2025

Quindi non ho ben capito, dobbiamo ingnorare questa PR a favore della #26 che condivide i primi 6 commit di questa?PR

Io non avendo accesso diretto ad un dispositivo IOS, non ho ancora potuto provare direttamente ne questa ne l'altra, non appena lo farò le farò sapere

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants