Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: whether the eapi return value is needs to be encrypted is controlled by the param ‘e_r’ (refer to NeteaseCloudMusicApi) #1454

Merged
merged 2 commits into from
Aug 10, 2024

Conversation

s12mmm3
Copy link
Contributor

@s12mmm3 s12mmm3 commented Aug 3, 2024

feat: whether the eapi return value is needs to be encrypted is controlled by the param ‘e_r’ (refer to NeteaseCloudMusicApi)

@1715173329 1715173329 requested a review from pan93412 August 4, 2024 05:12
@pan93412 pan93412 merged commit 11c1ed5 into UnblockNeteaseMusic:enhanced Aug 10, 2024
8 checks passed
@1715173329
Copy link
Member

I'm getting this error on latest PC client 3.0.2, could you check it?

ERROR: (hook) A error occurred in hook.request.before when hooking https://music.163.com/api/clientlog/encrypt/upload/sysaction?multiupload=true.
    URIError: URI malformed
        at decodeURIComponent (<anonymous>)
        at /home/cnsztl/project/UnblockNeteaseMusic/src/hook.js:223:8
        at process.processTicksAndRejections (node:internal/process/task_queues:95:5)

looks like the data is in binary format.

@s12mmm3
Copy link
Contributor Author

s12mmm3 commented Aug 31, 2024

It seems that isn`t all urls that started with ‘/api’ can be hooked;
Under normal conditions, there are only the urls started with '/eapi' in the client; Only by modifying the client config will urls starting with '/api' appear;
So for safety, not to process the urls started with '/api' by default.

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.

3 participants