google / google/GoogleSignIn-iOS
Add App Check support for macOS
- Lingua principale
- Objective-C
- Stelle
- 750
- Fork
- 282
- Merge medio
- 2g 15h
- PR unite (30g)
- 9
Descrizione
**Is your feature request related to a problem you're having? Please describe.**
App Check is currently not supported on macOS. The `GIDSignIn.sharedInstance.configure` method, which is required to enable App Check, is only available on iOS.
Looking at the source code, App Check related code is wrapped in `#if TARGET_OS_IOS && !TARGET_OS_MACCATALYST`, which confirms that macOS is intentionally excluded from the current implementation.
**Describe the solution you'd like**
Please add App Check support for macOS so that `GIDSignIn.sharedInstance.configure` (or an equivalent method) can be called on macOS as well.
**Describe alternatives you've considered**
No alternatives have been found. macOS apps are currently forced to run without App Check.
**Additional context**
Relevant source file: `GoogleSignIn/Sources/GIDSignIn.m`
```objc
#if TARGET_OS_IOS && !TARGET_OS_MACCATALYST
#import
#import "GoogleSignIn/Sources/GIDAppCheck/Implementations/GIDAppCheck.h"
// ...
#endif // TARGET_OS_IOS && !TARGET_OS_MACCATALYST
```
Guida per i contributori
Apri la guida per i contributori
Direzione di ricerca
Inizia in GoogleSignIn/Sources/GIDSignIn.m e segui il codice App Check protetto da TARGET_OS_IOS \u0026\u0026 !TARGET_OS_MACCATALYST, incluso il percorso GIDSignIn.sharedInstance.configure. Confronta le condizioni di piattaforma esistenti e determina il punto di ingresso compatibile con macOS. Il lavoro è completato quando le app macOS possono abilitare App Check tramite GIDSignIn.sharedInstance.configure o un metodo equivalente.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- objective-c
- Ambito
- authentication, desktop
- Tipo di issue
- Funzionalità
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Stato di attività
- Tranquilla
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 52/100