healthcheck fails if apple id has underscore in it about docker-icloudpd HOT 4 CLOSED

local quick fix for me: changed regex to include underscore
sync-icloud.sh:13 and healthcheck.sh:19:
changed ${apple_id//[^a-zA-Z0-9]/} to ${apple_id//[^a-zA-Z0-9_]/}

because i couldn't find the reason why the underscore is not filtered at file creation

from docker-icloudpd.

Hmmmm... This sounds a bit odd to be honest.

I used to have an underscore in my Apple ID, so this did work once upon a time. I had to swap to a non-underscored ID when I setup family safety on my older devices as Apple don't support underscores in the e-mail address used for parent accounts...

However, main sync-icloud.sh script also uses the same parameter expansion to calculate the cookie's filename (line 13).

This is used a lot throughout the script, so if you have an underscore in your address, I'd expect the main script to bomb out too.

Have you changed line 13 of the main script too?

from docker-icloudpd.

Yes i changed the regex in both files: sync-icloud.sh and healthcheck.sh

I suspect that maybe the core lib changed somehow.
I haven't looked that deep in the other repo after i fixed it for me locally.

from docker-icloudpd.

Ah, OK cool... I've amended the code so the change will appear when I next push the images to Dockerhub.

from docker-icloudpd.