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

1.2.4 release #2336

Closed
4 tasks done
sosnovsky opened this issue Aug 29, 2023 · 10 comments
Closed
4 tasks done

1.2.4 release #2336

sosnovsky opened this issue Aug 29, 2023 · 10 comments
Assignees

Comments

@sosnovsky
Copy link
Collaborator

sosnovsky commented Aug 29, 2023

  • update screenshots
  • release notes
  • upload build
  • send to review (on Sep 18)
@sosnovsky sosnovsky added this to the 1.2.5: Maintenance milestone Aug 29, 2023
@sosnovsky sosnovsky self-assigned this Aug 29, 2023
@sosnovsky sosnovsky changed the title 1.2.5 release 1.2.4 release Aug 31, 2023
@sosnovsky
Copy link
Collaborator Author

@tomholub @martgil @ioanmo226 new build 1.2.4 (1) is available on Testflight.
Let's test it on this week, and if all good - I'll send it to App Store review next Monday.

@martgil
Copy link
Collaborator

martgil commented Sep 6, 2023

Hello Roma, thank you for sending me the TestFlight invitation. I will let you know how my testing for SSL Pinning implementation goes. Thanks again.

@martgil
Copy link
Collaborator

martgil commented Sep 6, 2023

@sosnovsky It looks like the SSL certificate pinning works very well. The tool I use has an SSL bypass feature which does not work against the FlowCrypt iOS app. Is it okay if I send you a short video clip demonstrating that?

@sosnovsky
Copy link
Collaborator Author

@sosnovsky It looks like the SSL certificate pinning works very well. The tool I use has an SSL bypass feature which does not work against the FlowCrypt iOS app. Is it okay if I send you a short video clip demonstrating that?

Great! Yes, please send this video

@sosnovsky
Copy link
Collaborator Author

@martgil I checked video - pinning works well, however we implemented it only for flowcrypt.com, but it seems to work also for gmail domains too. Can you please also check it on current App Store version and send me the video? thanks

@ioanmo226 I just found that from iOS 14 it's possible to pin certificates just by using NSPinnedDomains config property - https://blogs.sap.com/2021/05/28/certificate-pinning-on-ios-14/, it looks much simpler that our current implementation. It also makes pinning additional domains easier. What do you think about using it?

@ioanmo226
Copy link
Collaborator

@sosnovsky, just wrapped up my review.
I think NSPinnedDomains is kinda rigid—lacks customization and is a not easy to debug.

On the flip side, URLSessionDelegate is way more flexible and easier to debug(a bit hard to implement though).
But yeah, NSPinnedDomains is quick to set up and is declarative.

What do you think?

@ioanmo226
Copy link
Collaborator

@martgil Please send me video as well(old one and new one too)

@sosnovsky
Copy link
Collaborator Author

I think NSPinnedDomains is kinda rigid—lacks customization and is a not easy to debug.

For debugging you still need to use some 3rd-party tools, as you can't check SSL pinning implementation directly in Xcode. So I think it's not a big disadvantage.

I was able to test it on Mac using Proxyman (guide about SSL Proxying - https://tanaschita.com/20220704-quick-guide-on-proxyman/). Current implementation works well, but error message is just cancelled instead of SSL certificate mismatch

Let's try to use NSPinnedDomains and fix this error message, maybe it'll be even easier with NSPinnedDomains

@ioanmo226
Copy link
Collaborator

OK

@sosnovsky
Copy link
Collaborator Author

Build 1.2.4 was published to App Store

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants