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

Update Starscream to v4.0.8 #1487

Closed
wants to merge 1 commit into from
Closed

Update Starscream to v4.0.8 #1487

wants to merge 1 commit into from

Conversation

gin0606
Copy link

@gin0606 gin0606 commented Mar 22, 2024

ref #1478

We need to update the version of Starscream to support the Privacy Manifest.

daltoniam/Starscream@4.0.6...4.0.8

This pull request is a redo of #1486.

I have reconfirmed the operation, and it appears to be functioning correctly in my application( socket.io 2 servers ).

@slice-varundudeja
Copy link

when will this get released? its needed to remove apple warnings.

@bmt2018
Copy link

bmt2018 commented Apr 4, 2024

With the privacy manifest deadline looming, if there isn't a new release that merges these changes for the Starscream dependency and you're using SPM, it looks like it might(?) be possible to fix this by bringing in the Socket.IO client as a local package and then editing the package.swift file to specify Starscream 4.0.8 instead of 4.0.6. I'm basing this assumption on https://developer.apple.com/documentation/xcode/editing-a-package-dependency-as-a-local-package. Has anyone tried this?

@amuyu
Copy link

amuyu commented Apr 9, 2024

Starscream needs to be updated because it is included in the following content.
I'm curious about when it will be reflected. (cc. @nuclearace )

The following are commonly used SDKs in apps on the App Store. Starting in spring 2024, you must include the privacy manifest for any SDK listed below when you submit new apps in App Store Connect that include those SDKs
https://developer.apple.com/support/third-party-SDK-requirements/

daltoniam/Starscream#1008

@cbiggin
Copy link

cbiggin commented Apr 11, 2024

This really does need to be updated because you've specified Exact Version (of Starscream v4.0.6) instead of Up to Next Major Version in the Package.swift. This means that I cannot even explicitly add Starscream v4.0.8 as an SPM package since it conflicts.

@WestFlow127
Copy link

@nuclearace this should be added to v16.1.1

@imnak0-naver
Copy link

is there anyone who can approve this PR? 'o'?

@@ -8,7 +8,7 @@ let package = Package(
.library(name: "SocketIO", targets: ["SocketIO"])
],
dependencies: [
.package(url: "https://github.com/daltoniam/Starscream", .exactItem("4.0.6")),
.package(url: "https://github.com/daltoniam/Starscream", .exactItem("4.0.8")),
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this should be upToNextMajor.

Copy link
Author

@gin0606 gin0606 Apr 18, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I also believe that's the way to go. However, since I don't fully understand the intentions behind the changes made in 3c81d0e , I feel it's not appropriate for me to do it. thx.

@EgzonArifi
Copy link

any updates on this?

@benjamin-es-hall
Copy link

The deadline requiring privacy manifests has now past and expecting releases including socket-io to get blocked by apple unless this is done. Any update on when this will be merged and deployed?

@tumata
Copy link

tumata commented Jun 6, 2024

@gin0606 Please note that updating Starscream to 4.0.8 while not also updating the podspec's s.ios.deployment_target = '12.0' leads to the following cocoapod error when building pods that rely on Socket.IO:

Socket.IO-Client-Swift/Source/SocketIO/Engine/SocketEngine.swift:27:8: error: compiling for iOS 11.0, but module 'Starscream' has a minimum deployment target of iOS 12.0

See #1488 - you can just merge that into this PR.

This is completely blocking our team...

@gin0606
Copy link
Author

gin0606 commented Jun 18, 2024

The changes in #1492 are better, so this Pull Request is no longer necessary.

@gin0606 gin0606 closed this Jun 18, 2024
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.

10 participants