diff options
| author | Fuwn <[email protected]> | 2026-02-23 08:20:28 -0800 |
|---|---|---|
| committer | Fuwn <[email protected]> | 2026-02-23 08:21:50 -0800 |
| commit | 13762193cf21624e6bab5e8f3c12ace320e39661 (patch) | |
| tree | b99599caa5062c2bf2f363168262e41ebc26b788 /Sora/Data | |
| parent | chore: deduplicate project build settings in xcodegen spec (diff) | |
| download | sora-testing-13762193cf21624e6bab5e8f3c12ace320e39661.tar.xz sora-testing-13762193cf21624e6bab5e8f3c12ace320e39661.zip | |
fix: use Danbooru login/api_key query auth
Diffstat (limited to 'Sora/Data')
| -rw-r--r-- | Sora/Data/Booru/BooruManager.swift | 14 | ||||
| -rw-r--r-- | Sora/Data/Booru/Provider/BooruProviderCredentials.swift | 36 | ||||
| -rw-r--r-- | Sora/Data/Settings/SettingsManager.swift | 21 |
3 files changed, 65 insertions, 6 deletions
diff --git a/Sora/Data/Booru/BooruManager.swift b/Sora/Data/Booru/BooruManager.swift index 562bb1e..2459f2d 100644 --- a/Sora/Data/Booru/BooruManager.swift +++ b/Sora/Data/Booru/BooruManager.swift @@ -295,10 +295,22 @@ class BooruManager: ObservableObject { // swiftlint:disable:this type_body_leng components.scheme = "https" components.host = domain components.path = "/posts.json" - components.queryItems = [ + + var queryItems = [ URLQueryItem(name: "page", value: String(page)), URLQueryItem(name: "tags", value: tagString), ] + + if let validCredentials = credentials { + let login = validCredentials.login.trimmingCharacters(in: .whitespacesAndNewlines) + + if !validCredentials.apiKey.isEmpty, !login.isEmpty { + queryItems.append(URLQueryItem(name: "login", value: login)) + queryItems.append(URLQueryItem(name: "api_key", value: validCredentials.apiKey)) + } + } + + components.queryItems = queryItems url = components.url case .moebooru: diff --git a/Sora/Data/Booru/Provider/BooruProviderCredentials.swift b/Sora/Data/Booru/Provider/BooruProviderCredentials.swift index 898201a..6733405 100644 --- a/Sora/Data/Booru/Provider/BooruProviderCredentials.swift +++ b/Sora/Data/Booru/Provider/BooruProviderCredentials.swift @@ -5,17 +5,45 @@ struct BooruProviderCredentials: Codable, Identifiable, Equatable { let provider: BooruProvider var apiKey: String var userID: Int + var login: String - init(provider: BooruProvider, apiKey: String, userID: Int, id: UUID = UUID()) { + init( + provider: BooruProvider, + apiKey: String, + userID: Int, + login: String = "", + id: UUID = UUID() + ) { self.id = id self.provider = provider self.apiKey = apiKey self.userID = userID + self.login = login + } + + // swiftlint:disable explicit_enum_raw_value + private enum CodingKeys: String, CodingKey { + case id + case provider + case apiKey + case userID + case login + } + // swiftlint:enable explicit_enum_raw_value + + init(from decoder: Decoder) throws { + let container = try decoder.container(keyedBy: CodingKeys.self) + + id = try container.decodeIfPresent(UUID.self, forKey: .id) ?? UUID() + provider = try container.decode(BooruProvider.self, forKey: .provider) + apiKey = try container.decodeIfPresent(String.self, forKey: .apiKey) ?? "" + userID = try container.decodeIfPresent(Int.self, forKey: .userID) ?? 0 + login = try container.decodeIfPresent(String.self, forKey: .login) ?? "" } static func from( // swiftlint:disable:next large_tuple - _ rawCredentials: [(provider: BooruProvider, apiKey: String, userID: Int)], + _ rawCredentials: [(provider: BooruProvider, apiKey: String, userID: Int, login: String)], existingCredentials: [Self] ) -> [Self] { rawCredentials.map { credentials in @@ -26,6 +54,7 @@ struct BooruProviderCredentials: Codable, Identifiable, Equatable { provider: credentials.provider, apiKey: credentials.apiKey, userID: credentials.userID, + login: credentials.login, id: existingKey.id ) } @@ -33,7 +62,8 @@ struct BooruProviderCredentials: Codable, Identifiable, Equatable { return Self( provider: credentials.provider, apiKey: credentials.apiKey, - userID: credentials.userID + userID: credentials.userID, + login: credentials.login ) } } diff --git a/Sora/Data/Settings/SettingsManager.swift b/Sora/Data/Settings/SettingsManager.swift index d7bfc44..ddeb1e4 100644 --- a/Sora/Data/Settings/SettingsManager.swift +++ b/Sora/Data/Settings/SettingsManager.swift @@ -289,7 +289,12 @@ class SettingsManager: ObservableObject { // swiftlint:disable:this type_body_l let existingCredentials: [BooruProviderCredentials] = Self.decode([BooruProviderCredentials].self, from: providerCredentialsData) ?? [] let rawCredentials = newValue.map { credentials in - (provider: credentials.provider, apiKey: credentials.apiKey, userID: credentials.userID) + ( + provider: credentials.provider, + apiKey: credentials.apiKey, + userID: credentials.userID, + login: credentials.login + ) } let mergedCredentials = BooruProviderCredentials.from( rawCredentials, existingCredentials: existingCredentials @@ -335,6 +340,13 @@ class SettingsManager: ObservableObject { // swiftlint:disable:this type_body_l ) } + var providerLogins: [BooruProvider: String] { + mergedCredentialValues( + extract: { $0.login }, + isDefault: { $0.trimmingCharacters(in: .whitespacesAndNewlines).isEmpty } + ) + } + // MARK: - Initialisation init() { syncObservation = NotificationCenter.default.addObserver( @@ -456,7 +468,12 @@ class SettingsManager: ObservableObject { // swiftlint:disable:this type_body_l let existingCredentials: [BooruProviderCredentials] = Self.decode([BooruProviderCredentials].self, from: providerCredentialsData) ?? [] let rawCredentials = newValue.map { credentials in - (provider: credentials.provider, apiKey: credentials.apiKey, userID: credentials.userID) + ( + provider: credentials.provider, + apiKey: credentials.apiKey, + userID: credentials.userID, + login: credentials.login + ) } let mergedCredentials = BooruProviderCredentials.from( rawCredentials, existingCredentials: existingCredentials |