search

TWCable / grabbit

Grabbit - Fast Content Sync tool for AEM/CQ
Apache License 2.0
125 stars 64 forks source link

Add authorization header preemptively #194

Closed jbornemann closed 6 years ago

jbornemann commented 6 years ago

@sagarsane per #192

Thought I'd knock it out real quick.

jbornemann commented 6 years ago

@sagarsane thoughts on this? I need to get my hands on AEM again, so I can actually test things my self.

sagarsane commented 6 years ago

I haven't had a chance to test this yet .. but the change looks sound to me!

pgoodrich commented 6 years ago

@jbornemann We're in the process of testing this now but it will be a while as we check across 6.2 and 6.3 for author and publish

pgoodrich commented 6 years ago

@jbornemann @sagarsane +1 Confirmed this is working on 6.2 and 6.3 author and publish.

6.3 has the same pre-requisites as 6.2. If the Deserialization Firefall is active, it requires the Grabbit-Deserialization-Firewall-Configuration-1.0.zip package and to whitelist the Grabbit bundle until we move away from using the admin session.

pgoodrich commented 6 years ago

Confirmed working on 6.1 as well.

sagarsane commented 6 years ago

Thanks @pgoodrich !