Skip to content

iOS mDNS lookup fails until a dialog box is clicked #46704

Description

@zanderso

Splitting off from #41085.

With Catalina and iOS 13, iOS mDNS lookup is reported to fail until a prompt from macOS is approved to allow dart to accept incoming network connections.

This issue is to investigate and recommend a fix, or an error message from the flutter tool as appropriate.

@dnfield @christopherfujino @jmagman @tvolkert

Activity

  1. added
    c: regressionIt was better in the past than it is now
    platform-iosiOS applications specifically
    toolAffects the "flutter" command-line tool. See also t: labels.
    engineflutter/engine related. See also e: labels.
    customer: crowdAffects or could affect many people, though not necessarily a specific customer.
    a: annoyanceRepeatedly frustrating issues with non-experimental functionality
    on Dec 10, 2019
  2. dnfield commented on Dec 10, 2019

    @dnfield
    Contributor

    @christopherfujino and I were looking into this at one point. As I recall, our discussion was something like having the tool/doctor check if the current Dart binary had an exception allowed in the firewall or not. I don't think we had come up wiht a way to bypass the allow/deny box altogether though.

  3. dnfield commented on Dec 10, 2019

    @dnfield
    Contributor

    I don't believe this is a regression though, or an engine issue.. Removing those labels.

  4. removed
    engineflutter/engine related. See also e: labels.
    c: regressionIt was better in the past than it is now
    on Dec 10, 2019
  5. zanderso commented on Dec 10, 2019

    @zanderso
    MemberAuthor

    Adding back the regression label since vmservice connections used to work without doing this.

  6. dnfield commented on Dec 10, 2019

    @dnfield
    Contributor

    Fair enough - although that was only because of log-scanning. We always had this dialog pop up the first time you tried to run flutter attach using mDNS.

  7. christopherfujino commented on Dec 10, 2019

    @christopherfujino
    Contributor

    And the doctor check wasn't a perfect solution either, because it is still dependent on the user recognizing the dialog and accepting.

  8. jmagman commented on Dec 10, 2019

    @jmagman
    Member

    In https://storage.googleapis.com/flutter_infra/flutter/ac9391978e7c0693b75a82c219e059b6ffee35c4/dart-sdk-darwin-x64.zip (v1.12.13+hotfix.4) the dart binary isn't codesigned at all and doesn't have any networking entitlements like https://developer.apple.com/documentation/bundleresources/entitlements/com_apple_security_network_server

    $ codesign -d --entitlements -  dart-sdk/bin/dart
    dart-sdk/bin/dart: code object is not signed at all
    
  9. christopherfujino commented on Dec 10, 2019

    @christopherfujino
    Contributor

    Good catch @jmagman, working on this!

  10. christopherfujino commented on Dec 10, 2019

    @christopherfujino
    Contributor

    The next hotfix to beta will be codesigned, and have the additional networking entitlements which will resolve the dialog box popping up. The next dev release should also pick up the entitlements. I will close this issue when both the beta and dev releases are published.

  11. christopherfujino commented on Dec 11, 2019

    @christopherfujino
    Contributor

    This is fixed on beta and the new stable release. Dev release should be very soon...

  12. christopherfujino commented on Dec 13, 2019

    @christopherfujino
    Contributor

    This should now be fixed on dev, beta, and stable, due to the dart binary now being signed with network entitlements. The dialog will still pop up on master, however, due to #44372.

  13. christopherfujino commented on Mar 19, 2020

    @christopherfujino
    Contributor

    It looks like due to a bug in the codesigning script, the appropriate entitlements were not added. Re-opening pending a fix to the codesigning process.

  14. zanderso commented on Jun 18, 2020

    @zanderso
    MemberAuthor

    @christopherfujino If it's still needed, let's track the codesigning script bug in a separate issue.

  15. christopherfujino commented on Jun 18, 2020

    @christopherfujino
    Contributor

    Thanks @zanderso. This was indeed fixed and I forgot to double back and close this issue.

  16. github-actions commented on Aug 19, 2021

    @github-actions

    This thread has been automatically locked since there has not been any recent activity after it was closed. If you are still experiencing a similar issue, please open a new bug, including the output of flutter doctor -v and a minimal reproduction of the issue.

  17. locked as resolved and limited conversation to collaborators on Aug 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Labels

a: annoyanceRepeatedly frustrating issues with non-experimental functionalityc: regressionIt was better in the past than it is nowcustomer: crowdAffects or could affect many people, though not necessarily a specific customer.platform-iosiOS applications specificallytoolAffects the "flutter" command-line tool. See also t: labels.

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions