[PR #387] [MERGED] Fix memory leak due to strong reference of controller #617

New issue

Closed

opened 2026-03-03 16:49:44 +03:00 by kerem · 0 comments

kerem commented

2026-03-03 16:49:44 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/OAuthSwift/OAuthSwift/pull/387
Author: @Econa77
Created: 6/30/2017
Status: ✅ Merged
Merged: 6/30/2017
Merged by: @phimage

Base: master ← Head: feature/fix-leak

📝 Commits (1)

1db6538 Fix memory leak due to strong reference of controller

📊 Changes

1 file changed (+4 additions, -4 deletions)

View changed files

📝 Sources/OAuthSwiftURLHandlerType.swift (+4 -4)

📄 Description

SafariURLHandler strongly referenced the UIViewController and was not released. so fixed it.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/OAuthSwift/OAuthSwift/pull/387 **Author:** [@Econa77](https://github.com/Econa77) **Created:** 6/30/2017 **Status:** ✅ Merged **Merged:** 6/30/2017 **Merged by:** [@phimage](https://github.com/phimage) **Base:** `master` ← **Head:** `feature/fix-leak` --- ### 📝 Commits (1) - [`1db6538`](https://github.com/OAuthSwift/OAuthSwift/commit/1db6538e454a6f0aac962bd30e7b3bb355513652) Fix memory leak due to strong reference of controller ### 📊 Changes **1 file changed** (+4 additions, -4 deletions) <details> <summary>View changed files</summary> 📝 `Sources/OAuthSwiftURLHandlerType.swift` (+4 -4) </details> ### 📄 Description SafariURLHandler strongly referenced the UIViewController and was not released. so fixed it. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>