firebase / firebase-ios-sdk

Firebase SDK for Apple App Development
https://firebase.google.com
Apache License 2.0
5.71k stars 1.49k forks source link

Insecure SSL: Overly Broad Certificate Trust #6821

Closed Shakihassan closed 4 years ago

Shakihassan commented 4 years ago

The call to dataTaskWithRequest:completionHandler:() in FIRInstanceIDTokenDeleteOperation.m on line 87 initiates an SSL/TLS connection using the default pre-loaded system Certificate Authorities (CAs) that might enable attackers to intercept encrypted communications by performing man-in-the-middle (MiTM) attacks using certificates signed with compromised root CAs.An SSL/TLS connection is created using the default pre-loaded system Certificate Authorities (CAs), that might enable attackers to intercept encrypted communications by performing man-in-the-middle (MiTM) attacks using certificates signed with compromised root CAs.

The call to dataTaskWithRequest:completionHandler:() in FIRInstanceIDTokenFetchOperation.m on line 120 initiates an SSL/TLS connection using the default pre-loaded system Certificate Authorities (CAs) that might enable attackers to intercept encrypted communications by performing man-in-the-middle (MiTM) attacks using certificates signed with compromised root CAs.An SSL/TLS connection is created using the default pre-loaded system Certificate Authorities (CAs), that might enable attackers to intercept encrypted communications by performing man-in-the-middle (MiTM) attacks using certificates signed with compromised root CAs.

Note : Reported by the Fortify on demand Team

google-oss-bot commented 4 years ago

I found a few problems with this issue:

morganchen12 commented 4 years ago

Firebase doesn't currently allow for custom certificate pinning on iOS. If you would like to request this feature, please file a feature request.