fix: send auth bearer token when browsing by collection without waiti…

…ng for 401

resolves #416

data-android/src/main/java/ly/david/data/di/logging/LoggingModule.kt

@@ -7,6 +7,10 @@ import timber.log.Timber
 val loggingModule = module {
     single<Logger> {
         object : Logger {
+            override fun d(text: String) {
+                Timber.d(text)
+            }
+
             override fun e(exception: Exception) {
                 Timber.e(exception)
             }

data/core/src/commonMain/kotlin/ly/david/data/core/logging/Logger.kt

@@ -1,5 +1,6 @@
 package ly.david.data.core.logging
 
 interface Logger {
+    fun d(text: String)
     fun e(exception: Exception)
 }

data/musicbrainz/src/commonMain/kotlin/ly/david/data/musicbrainz/api/MusicBrainzApi.kt

@@ -31,16 +31,14 @@ interface MusicBrainzApi : SearchApi, BrowseApi, LookupApi, CollectionApi, Music
                                 BearerTokens(it, "")
                             }
                         }
-                        // TODO: this block is never executed unlike for spotify
                         refreshTokens {
                             musicBrainzAuthRepository.getAuthState()?.let {
                                 BearerTokens(it, "")
                             }
                         }
-                        // TODO: handle collection browse, one way to do it is to split up the
-                        //  api that requires auth and just return true here
                         sendWithoutRequest { request ->
-                            request.url.pathSegments.contains(USER_INFO)
+                            request.url.parameters["collection"] != null ||
+                                request.url.pathSegments.contains(USER_INFO)
                         }
                     }
                 }