[4.x] Avoid exposing internal functions and properties (#8137)

* Avoid exposing internal functions and properties

This is a backport for 4.x branch from f408411ff9.

* More internal modifiers

mockwebserver/src/main/kotlin/okhttp3/mockwebserver/MockResponse.kt

@@ -13,6 +13,7 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
+@file:Suppress("INVISIBLE_MEMBER", "INVISIBLE_REFERENCE")
 package okhttp3.mockwebserver
 
 import java.util.concurrent.TimeUnit

mockwebserver/src/main/kotlin/okhttp3/mockwebserver/MockWebServer.kt

@@ -15,6 +15,7 @@
  * limitations under the License.
  */
 
+@file:Suppress("INVISIBLE_MEMBER", "INVISIBLE_REFERENCE")
 package okhttp3.mockwebserver
 
 import java.io.Closeable

okcurl/src/main/kotlin/okhttp3/curl/Main.kt

@@ -36,7 +36,6 @@ import okhttp3.Protocol
 import okhttp3.Request
 import okhttp3.RequestBody.Companion.toRequestBody
 import okhttp3.curl.Main.Companion.NAME
-import okhttp3.internal.format
 import okhttp3.internal.http.StatusLine
 import okhttp3.internal.http2.Http2
 import okhttp3.internal.platform.Platform
@@ -48,6 +47,7 @@ import picocli.CommandLine.Command
 import picocli.CommandLine.IVersionProvider
 import picocli.CommandLine.Option
 import picocli.CommandLine.Parameters
+import java.util.Locale
 
 @Command(name = NAME, description = ["A curl for the next-generation web."],
     mixinStandardHelpOptions = true, versionProvider = Main.VersionProvider::class)
@@ -273,7 +273,7 @@ class Main : Runnable {
           level = Level.FINE
           formatter = object : SimpleFormatter() {
             override fun format(record: LogRecord): String {
-              return format("%s%n", record.message)
+              return "%s%n".format(Locale.US, record.message)
             }
           }
         })
@@ -289,9 +289,9 @@ class Main : Runnable {
             override fun format(record: LogRecord): String {
               val parameters = record.parameters
               if (parameters != null) {
-                return format("%s%n%s%n", record.message, record.parameters.first())
+                return "%s%n%s%n".format(Locale.US, record.message, record.parameters.first())
               } else {
-                return format("%s%n", record.message)
+                return "%s%n".format(Locale.US, record.message)
               }
             }
           }

okhttp-sse/src/main/kotlin/okhttp3/internal/sse/RealEventSource.kt

@@ -13,6 +13,7 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
+@file:Suppress("INVISIBLE_MEMBER", "INVISIBLE_REFERENCE")
 package okhttp3.internal.sse
 
 import java.io.IOException

okhttp-sse/src/main/kotlin/okhttp3/internal/sse/ServerSentEventReader.kt

@@ -13,6 +13,7 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
+@file:Suppress("INVISIBLE_MEMBER", "INVISIBLE_REFERENCE")
 package okhttp3.internal.sse
 
 import java.io.IOException

okhttp-tls/src/main/kotlin/okhttp3/tls/HandshakeCertificates.kt

@@ -13,6 +13,7 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
+@file:Suppress("INVISIBLE_MEMBER", "INVISIBLE_REFERENCE")
 package okhttp3.tls
 
 import java.security.SecureRandom

okhttp-tls/src/main/kotlin/okhttp3/tls/HeldCertificate.kt

@@ -13,6 +13,7 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
+@file:Suppress("INVISIBLE_MEMBER", "INVISIBLE_REFERENCE")
 package okhttp3.tls
 
 import java.math.BigInteger

okhttp-tls/src/main/kotlin/okhttp3/tls/internal/InsecureExtendedTrustManager.kt

@@ -13,6 +13,7 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
+@file:Suppress("INVISIBLE_MEMBER", "INVISIBLE_REFERENCE")
 package okhttp3.tls.internal
 
 import java.net.Socket

okhttp-urlconnection/src/main/kotlin/okhttp3/JavaNetCookieJar.kt

@@ -13,6 +13,7 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
+@file:Suppress("INVISIBLE_MEMBER", "INVISIBLE_REFERENCE")
 package okhttp3
 
 import java.io.IOException

okhttp/src/main/kotlin/okhttp3/internal/Util.kt

@@ -57,14 +57,14 @@ import okio.Options
 import okio.Source
 
 @JvmField
-val EMPTY_BYTE_ARRAY = ByteArray(0)
+internal val EMPTY_BYTE_ARRAY = ByteArray(0)
 @JvmField
-val EMPTY_HEADERS = headersOf()
+internal val EMPTY_HEADERS = headersOf()
 
 @JvmField
-val EMPTY_RESPONSE = EMPTY_BYTE_ARRAY.toResponseBody()
+internal val EMPTY_RESPONSE = EMPTY_BYTE_ARRAY.toResponseBody()
 @JvmField
-val EMPTY_REQUEST = EMPTY_BYTE_ARRAY.toRequestBody()
+internal val EMPTY_REQUEST = EMPTY_BYTE_ARRAY.toRequestBody()
 
 /** Byte order marks. */
 private val UNICODE_BOMS = Options.of(
@@ -77,7 +77,7 @@ private val UNICODE_BOMS = Options.of(
 
 /** GMT and UTC are equivalent for our purposes. */
 @JvmField
-val UTC = TimeZone.getTimeZone("GMT")!!
+internal val UTC = TimeZone.getTimeZone("GMT")!!
 
 /**
  * Quick and dirty pattern to differentiate IP addresses from hostnames. This is an approximation
@@ -92,13 +92,13 @@ val UTC = TimeZone.getTimeZone("GMT")!!
 private val VERIFY_AS_IP_ADDRESS =
     "([0-9a-fA-F]*:[0-9a-fA-F:.]*)|([\\d.]+)".toRegex()
 
-fun checkOffsetAndCount(arrayLength: Long, offset: Long, count: Long) {
+internal fun checkOffsetAndCount(arrayLength: Long, offset: Long, count: Long) {
   if (offset or count < 0L || offset > arrayLength || arrayLength - offset < count) {
     throw ArrayIndexOutOfBoundsException()
   }
 }
 
-fun threadFactory(
+internal fun threadFactory(
   name: String,
   daemon: Boolean
 ): ThreadFactory = ThreadFactory { runnable ->
@@ -111,7 +111,7 @@ fun threadFactory(
  * Returns an array containing only elements found in this array and also in [other]. The returned
  * elements are in the same order as in this.
  */
-fun Array<String>.intersect(
+internal fun Array<String>.intersect(
   other: Array<String>,
   comparator: Comparator<in String>
 ): Array<String> {
@@ -133,7 +133,7 @@ fun Array<String>.intersect(
  * likelihood of an intersection so great, that it is not worth the CPU cost of sorting or the
  * memory cost of hashing.
  */
-fun Array<String>.hasIntersection(
+internal fun Array<String>.hasIntersection(
   other: Array<String>?,
   comparator: Comparator<in String>
 ): Boolean {
@@ -150,7 +150,7 @@ fun Array<String>.hasIntersection(
   return false
 }
 
-fun HttpUrl.toHostHeader(includeDefaultPort: Boolean = false): String {
+internal fun HttpUrl.toHostHeader(includeDefaultPort: Boolean = false): String {
   val host = if (":" in host) {
     "[$host]"
   } else {
@@ -163,11 +163,11 @@ fun HttpUrl.toHostHeader(includeDefaultPort: Boolean = false): String {
   }
 }
 
-fun Array<String>.indexOf(value: String, comparator: Comparator<String>): Int =
+internal fun Array<String>.indexOf(value: String, comparator: Comparator<String>): Int =
     indexOfFirst { comparator.compare(it, value) == 0 }
 
 @Suppress("UNCHECKED_CAST")
-fun Array<String>.concat(value: String): Array<String> {
+internal fun Array<String>.concat(value: String): Array<String> {
   val result = copyOf(size + 1)
   result[result.lastIndex] = value
   return result as Array<String>
@@ -176,7 +176,7 @@ fun Array<String>.concat(value: String): Array<String> {
 /**
  * Increments [startIndex] until this string is not ASCII whitespace. Stops at [endIndex].
  */
-fun String.indexOfFirstNonAsciiWhitespace(startIndex: Int = 0, endIndex: Int = length): Int {
+internal fun String.indexOfFirstNonAsciiWhitespace(startIndex: Int = 0, endIndex: Int = length): Int {
   for (i in startIndex until endIndex) {
     when (this[i]) {
       '\t', '\n', '\u000C', '\r', ' ' -> Unit
@@ -189,7 +189,7 @@ fun String.indexOfFirstNonAsciiWhitespace(startIndex: Int = 0, endIndex: Int = l
 /**
  * Decrements [endIndex] until `input[endIndex - 1]` is not ASCII whitespace. Stops at [startIndex].
  */
-fun String.indexOfLastNonAsciiWhitespace(startIndex: Int = 0, endIndex: Int = length): Int {
+internal fun String.indexOfLastNonAsciiWhitespace(startIndex: Int = 0, endIndex: Int = length): Int {
   for (i in endIndex - 1 downTo startIndex) {
     when (this[i]) {
       '\t', '\n', '\u000C', '\r', ' ' -> Unit
@@ -200,7 +200,7 @@ fun String.indexOfLastNonAsciiWhitespace(startIndex: Int = 0, endIndex: Int = le
 }
 
 /** Equivalent to `string.substring(startIndex, endIndex).trim()`. */
-fun String.trimSubstring(startIndex: Int = 0, endIndex: Int = length): String {
+internal fun String.trimSubstring(startIndex: Int = 0, endIndex: Int = length): String {
   val start = indexOfFirstNonAsciiWhitespace(startIndex, endIndex)
   val end = indexOfLastNonAsciiWhitespace(start, endIndex)
   return substring(start, end)
@@ -210,7 +210,7 @@ fun String.trimSubstring(startIndex: Int = 0, endIndex: Int = length): String {
  * Returns the index of the first character in this string that contains a character in
  * [delimiters]. Returns endIndex if there is no such character.
  */
-fun String.delimiterOffset(delimiters: String, startIndex: Int = 0, endIndex: Int = length): Int {
+internal fun String.delimiterOffset(delimiters: String, startIndex: Int = 0, endIndex: Int = length): Int {
   for (i in startIndex until endIndex) {
     if (this[i] in delimiters) return i
   }
@@ -221,7 +221,7 @@ fun String.delimiterOffset(delimiters: String, startIndex: Int = 0, endIndex: In
  * Returns the index of the first character in this string that is [delimiter]. Returns [endIndex]
  * if there is no such character.
  */
-fun String.delimiterOffset(delimiter: Char, startIndex: Int = 0, endIndex: Int = length): Int {
+internal fun String.delimiterOffset(delimiter: Char, startIndex: Int = 0, endIndex: Int = length): Int {
   for (i in startIndex until endIndex) {
     if (this[i] == delimiter) return i
   }
@@ -232,7 +232,7 @@ fun String.delimiterOffset(delimiter: Char, startIndex: Int = 0, endIndex: Int =
  * Returns the index of the first character in this string that is either a control character (like
  * `\u0000` or `\n`) or a non-ASCII character. Returns -1 if this string has no such characters.
  */
-fun String.indexOfControlOrNonAscii(): Int {
+internal fun String.indexOfControlOrNonAscii(): Int {
   for (i in 0 until length) {
     val c = this[i]
     if (c <= '\u001f' || c >= '\u007f') {
@@ -243,12 +243,12 @@ fun String.indexOfControlOrNonAscii(): Int {
 }
 
 /** Returns true if this string is not a host name and might be an IP address. */
-fun String.canParseAsIpAddress(): Boolean {
+internal fun String.canParseAsIpAddress(): Boolean {
   return VERIFY_AS_IP_ADDRESS.matches(this)
 }
 
 /** Returns true if we should void putting this this header in an exception or toString(). */
-fun isSensitiveHeader(name: String): Boolean {
+internal fun isSensitiveHeader(name: String): Boolean {
   return name.equals("Authorization", ignoreCase = true) ||
       name.equals("Cookie", ignoreCase = true) ||
       name.equals("Proxy-Authorization", ignoreCase = true) ||
@@ -256,12 +256,12 @@ fun isSensitiveHeader(name: String): Boolean {
 }
 
 /** Returns a [Locale.US] formatted [String]. */
-fun format(format: String, vararg args: Any): String {
+internal fun format(format: String, vararg args: Any): String {
   return String.format(Locale.US, format, *args)
 }
 
 @Throws(IOException::class)
-fun BufferedSource.readBomAsCharset(default: Charset): Charset {
+internal fun BufferedSource.readBomAsCharset(default: Charset): Charset {
   return when (select(UNICODE_BOMS)) {
     0 -> UTF_8
     1 -> UTF_16BE
@@ -273,7 +273,7 @@ fun BufferedSource.readBomAsCharset(default: Charset): Charset {
   }
 }
 
-fun checkDuration(name: String, duration: Long, unit: TimeUnit?): Int {
+internal fun checkDuration(name: String, duration: Long, unit: TimeUnit?): Int {
   check(duration >= 0L) { "$name < 0" }
   check(unit != null) { "unit == null" }
   val millis = unit.toMillis(duration)
@@ -282,14 +282,14 @@ fun checkDuration(name: String, duration: Long, unit: TimeUnit?): Int {
   return millis.toInt()
 }
 
-fun Char.parseHexDigit(): Int = when (this) {
+internal fun Char.parseHexDigit(): Int = when (this) {
   in '0'..'9' -> this - '0'
   in 'a'..'f' -> this - 'a' + 10
   in 'A'..'F' -> this - 'A' + 10
   else -> -1
 }
 
-fun List<Header>.toHeaders(): Headers {
+internal fun List<Header>.toHeaders(): Headers {
   val builder = Headers.Builder()
   for ((name, value) in this) {
     builder.addLenient(name.utf8(), value.utf8())
@@ -297,30 +297,30 @@ fun List<Header>.toHeaders(): Headers {
   return builder.build()
 }
 
-fun Headers.toHeaderList(): List<Header> = (0 until size).map {
+internal fun Headers.toHeaderList(): List<Header> = (0 until size).map {
   Header(name(it), value(it))
 }
 
 /** Returns true if an HTTP request for this URL and [other] can reuse a connection. */
-fun HttpUrl.canReuseConnectionFor(other: HttpUrl): Boolean = host == other.host &&
+internal fun HttpUrl.canReuseConnectionFor(other: HttpUrl): Boolean = host == other.host &&
     port == other.port &&
     scheme == other.scheme
 
-fun EventListener.asFactory() = EventListener.Factory { this }
+internal fun EventListener.asFactory() = EventListener.Factory { this }
 
-infix fun Byte.and(mask: Int): Int = toInt() and mask
-infix fun Short.and(mask: Int): Int = toInt() and mask
-infix fun Int.and(mask: Long): Long = toLong() and mask
+internal infix fun Byte.and(mask: Int): Int = toInt() and mask
+internal infix fun Short.and(mask: Int): Int = toInt() and mask
+internal infix fun Int.and(mask: Long): Long = toLong() and mask
 
 @Throws(IOException::class)
-fun BufferedSink.writeMedium(medium: Int) {
+internal fun BufferedSink.writeMedium(medium: Int) {
   writeByte(medium.ushr(16) and 0xff)
   writeByte(medium.ushr(8) and 0xff)
   writeByte(medium and 0xff)
 }
 
 @Throws(IOException::class)
-fun BufferedSource.readMedium(): Int {
+internal fun BufferedSource.readMedium(): Int {
   return (readByte() and 0xff shl 16
       or (readByte() and 0xff shl 8)
       or (readByte() and 0xff))
@@ -331,7 +331,7 @@ fun BufferedSource.readMedium(): Int {
  * deadline if one exists already.
  */
 @Throws(IOException::class)
-fun Source.skipAll(duration: Int, timeUnit: TimeUnit): Boolean {
+internal fun Source.skipAll(duration: Int, timeUnit: TimeUnit): Boolean {
   val nowNs = System.nanoTime()
   val originalDurationNs = if (timeout().hasDeadline()) {
     timeout().deadlineNanoTime() - nowNs
@@ -361,13 +361,13 @@ fun Source.skipAll(duration: Int, timeUnit: TimeUnit): Boolean {
  * source is helpful, such as when doing so completes a cache body or frees a socket connection for
  * reuse.
  */
-fun Source.discard(timeout: Int, timeUnit: TimeUnit): Boolean = try {
+internal fun Source.discard(timeout: Int, timeUnit: TimeUnit): Boolean = try {
   this.skipAll(timeout, timeUnit)
 } catch (_: IOException) {
   false
 }
 
-fun Socket.peerName(): String {
+internal fun Socket.peerName(): String {
   val address = remoteSocketAddress
   return if (address is InetSocketAddress) address.hostName else address.toString()
 }
@@ -381,7 +381,7 @@ fun Socket.peerName(): String {
  *
  * @param source the source used to read bytes from the socket.
  */
-fun Socket.isHealthy(source: BufferedSource): Boolean {
+internal fun Socket.isHealthy(source: BufferedSource): Boolean {
   return try {
     val readTimeout = soTimeout
     try {
@@ -398,14 +398,14 @@ fun Socket.isHealthy(source: BufferedSource): Boolean {
 }
 
 /** Run [block] until it either throws an [IOException] or completes. */
-inline fun ignoreIoExceptions(block: () -> Unit) {
+internal inline fun ignoreIoExceptions(block: () -> Unit) {
   try {
     block()
   } catch (_: IOException) {
   }
 }
 
-inline fun threadName(name: String, block: () -> Unit) {
+internal inline fun threadName(name: String, block: () -> Unit) {
   val currentThread = Thread.currentThread()
   val oldName = currentThread.name
   currentThread.name = name
@@ -416,7 +416,7 @@ inline fun threadName(name: String, block: () -> Unit) {
   }
 }
 
-fun Buffer.skipAll(b: Byte): Int {
+internal fun Buffer.skipAll(b: Byte): Int {
   var count = 0
   while (!exhausted() && this[0] == b) {
     count++
@@ -429,7 +429,7 @@ fun Buffer.skipAll(b: Byte): Int {
  * Returns the index of the next non-whitespace character in this. Result is undefined if input
  * contains newline characters.
  */
-fun String.indexOfNonWhitespace(startIndex: Int = 0): Int {
+internal fun String.indexOfNonWhitespace(startIndex: Int = 0): Int {
   for (i in startIndex until length) {
     val c = this[i]
     if (c != ' ' && c != '\t') {
@@ -440,11 +440,11 @@ fun String.indexOfNonWhitespace(startIndex: Int = 0): Int {
 }
 
 /** Returns the Content-Length as reported by the response headers. */
-fun Response.headersContentLength(): Long {
+internal fun Response.headersContentLength(): Long {
   return headers["Content-Length"]?.toLongOrDefault(-1L) ?: -1L
 }
 
-fun String.toLongOrDefault(defaultValue: Long): Long {
+internal fun String.toLongOrDefault(defaultValue: Long): Long {
   return try {
     toLong()
   } catch (_: NumberFormatException) {
@@ -456,7 +456,7 @@ fun String.toLongOrDefault(defaultValue: Long): Long {
  * Returns this as a non-negative integer, or 0 if it is negative, or [Int.MAX_VALUE] if it is too
  * large, or [defaultValue] if it cannot be parsed.
  */
-fun String?.toNonNegativeInt(defaultValue: Int): Int {
+internal fun String?.toNonNegativeInt(defaultValue: Int): Int {
   try {
     val value = this?.toLong() ?: return defaultValue
     return when {
@@ -470,18 +470,18 @@ fun String?.toNonNegativeInt(defaultValue: Int): Int {
 }
 
 /** Returns an immutable copy of this. */
-fun <T> List<T>.toImmutableList(): List<T> {
+internal fun <T> List<T>.toImmutableList(): List<T> {
   return Collections.unmodifiableList(toMutableList())
 }
 
 /** Returns an immutable list containing [elements]. */
 @SafeVarargs
-fun <T> immutableListOf(vararg elements: T): List<T> {
+internal fun <T> immutableListOf(vararg elements: T): List<T> {
   return Collections.unmodifiableList(listOf(*elements.clone()))
 }
 
 /** Returns an immutable copy of this. */
-fun <K, V> Map<K, V>.toImmutableMap(): Map<K, V> {
+internal fun <K, V> Map<K, V>.toImmutableMap(): Map<K, V> {
   return if (isEmpty()) {
     emptyMap()
   } else {
@@ -490,7 +490,7 @@ fun <K, V> Map<K, V>.toImmutableMap(): Map<K, V> {
 }
 
 /** Closes this, ignoring any checked exceptions. */
-fun Closeable.closeQuietly() {
+internal fun Closeable.closeQuietly() {
   try {
     close()
   } catch (rethrown: RuntimeException) {
@@ -500,7 +500,7 @@ fun Closeable.closeQuietly() {
 }
 
 /** Closes this, ignoring any checked exceptions. */
-fun Socket.closeQuietly() {
+internal fun Socket.closeQuietly() {
   try {
     close()
   } catch (e: AssertionError) {
@@ -517,7 +517,7 @@ fun Socket.closeQuietly() {
 }
 
 /** Closes this, ignoring any checked exceptions.  */
-fun ServerSocket.closeQuietly() {
+internal fun ServerSocket.closeQuietly() {
   try {
     close()
   } catch (rethrown: RuntimeException) {
@@ -537,7 +537,7 @@ fun ServerSocket.closeQuietly() {
  *
  * @param file a file in the directory to check. This file shouldn't already exist!
  */
-fun FileSystem.isCivilized(file: File): Boolean {
+internal fun FileSystem.isCivilized(file: File): Boolean {
   sink(file).use {
     try {
       delete(file)
@@ -549,20 +549,20 @@ fun FileSystem.isCivilized(file: File): Boolean {
   return false
 }
 
-fun Long.toHexString(): String = java.lang.Long.toHexString(this)
+internal fun Long.toHexString(): String = java.lang.Long.toHexString(this)
 
-fun Int.toHexString(): String = Integer.toHexString(this)
+internal fun Int.toHexString(): String = Integer.toHexString(this)
 
 @Suppress("PLATFORM_CLASS_MAPPED_TO_KOTLIN", "NOTHING_TO_INLINE")
-inline fun Any.wait() = (this as Object).wait()
+internal inline fun Any.wait() = (this as Object).wait()
 
 @Suppress("PLATFORM_CLASS_MAPPED_TO_KOTLIN", "NOTHING_TO_INLINE")
-inline fun Any.notify() = (this as Object).notify()
+internal inline fun Any.notify() = (this as Object).notify()
 
 @Suppress("PLATFORM_CLASS_MAPPED_TO_KOTLIN", "NOTHING_TO_INLINE")
-inline fun Any.notifyAll() = (this as Object).notifyAll()
+internal inline fun Any.notifyAll() = (this as Object).notifyAll()
 
-fun <T> readFieldOrNull(instance: Any, fieldType: Class<T>, fieldName: String): T? {
+internal fun <T> readFieldOrNull(instance: Any, fieldType: Class<T>, fieldName: String): T? {
   var c: Class<*> = instance.javaClass
   while (c != Any::class.java) {
     try {
@@ -617,11 +617,11 @@ internal inline fun Any.assertThreadDoesntHoldLock() {
   }
 }
 
-fun Exception.withSuppressed(suppressed: List<Exception>): Throwable = apply {
+internal fun Exception.withSuppressed(suppressed: List<Exception>): Throwable = apply {
   for (e in suppressed) addSuppressed(e)
 }
 
-inline fun <T> Iterable<T>.filterList(predicate: T.() -> Boolean): List<T> {
+internal inline fun <T> Iterable<T>.filterList(predicate: T.() -> Boolean): List<T> {
   var result: List<T> = emptyList()
   for (i in this) {
     if (predicate(i)) {

okhttp/src/main/kotlin/okhttp3/internal/hostnames.kt

@@ -29,7 +29,7 @@ import okio.Buffer
  * `null` will be returned if the host cannot be ToASCII encoded or if the result contains
  * unsupported ASCII characters.
  */
-fun String.toCanonicalHost(): String? {
+internal fun String.toCanonicalHost(): String? {
   val host: String = this
 
   // If the input contains a :, it’s an IPv6 address.

okhttp/src/main/kotlin/okhttp3/internal/internal.kt

@@ -27,19 +27,19 @@ import okhttp3.Headers
 import okhttp3.HttpUrl
 import okhttp3.Request
 
-fun parseCookie(currentTimeMillis: Long, url: HttpUrl, setCookie: String): Cookie? =
+internal fun parseCookie(currentTimeMillis: Long, url: HttpUrl, setCookie: String): Cookie? =
     Cookie.parse(currentTimeMillis, url, setCookie)
 
-fun cookieToString(cookie: Cookie, forObsoleteRfc2965: Boolean) =
+internal fun cookieToString(cookie: Cookie, forObsoleteRfc2965: Boolean) =
     cookie.toString(forObsoleteRfc2965)
 
-fun addHeaderLenient(builder: Headers.Builder, line: String) =
+internal fun addHeaderLenient(builder: Headers.Builder, line: String) =
     builder.addLenient(line)
 
-fun addHeaderLenient(builder: Headers.Builder, name: String, value: String) =
+internal fun addHeaderLenient(builder: Headers.Builder, name: String, value: String) =
     builder.addLenient(name, value)
 
-fun cacheGet(cache: Cache, request: Request) = cache.get(request)
+internal fun cacheGet(cache: Cache, request: Request) = cache.get(request)
 
-fun applyConnectionSpec(connectionSpec: ConnectionSpec, sslSocket: SSLSocket, isFallback: Boolean) =
+internal fun applyConnectionSpec(connectionSpec: ConnectionSpec, sslSocket: SSLSocket, isFallback: Boolean) =
     connectionSpec.apply(sslSocket, isFallback)