-
Notifications
You must be signed in to change notification settings - Fork 472
Issue #696: browser-engine-system: Clear "X-Requested-With" header when loading URLs. #699
Conversation
e1ef449
to
e09f177
Compare
Codecov Report
@@ Coverage Diff @@
## master #699 +/- ##
============================================
+ Coverage 78.65% 78.66% +0.01%
Complexity 1125 1125
============================================
Files 158 158
Lines 4085 4087 +2
Branches 600 600
============================================
+ Hits 3213 3215 +2
Misses 578 578
Partials 294 294
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
2 nits. Otherwise, 🚢 !
@@ -29,12 +37,12 @@ class SystemEngineSession(private val defaultSettings: Settings? = null) : Engin | |||
val internalView = currentView() | |||
|
|||
if (internalView == null) { | |||
// We can't load a URL without a WebView. So let's just remember the URL here until | |||
// We can't load al URL without a WebView. So let's just remember the URL here until |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
typo?
@@ -13,6 +13,14 @@ import kotlinx.coroutines.experimental.launch | |||
import mozilla.components.concept.engine.Settings | |||
import mozilla.components.concept.engine.request.RequestInterceptor | |||
|
|||
internal val additionalHeaders = HashMap<String, String>().apply { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could use mutableMapOf/mapOf("X-Requested-With" to "")
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, that's nicer. I'll update :)
…ith" header when loading URLs.
e09f177
to
91038db
Compare
Yet another WebView thing.. :)
Closes #696