Feat(medusajs) Allow to pass custom headers

[adrien2p]

The request client allow to pass an argument which correspond to the custom headers that consumer would like to add to the request.

When using the wrapping of the resource, the ability to set those headers is lost.

As a consumer, using nextJS, and wanting to take advantage of getServerSideProps I am not able to set the headers coming from the request and therefore unable to send an authenticated request.
This is a huge constraint since the only other way is to rewrite the resources on my own just to add that parameters (which is already accepted by the request)

In order to fix that, the customHeaders has been added to all api method as an optional parameters (backward compatibility) and therefore a consumer is able to use the resrouces even in getServerSideProps which allow a big performance boost

[adrien2p]

@oliverheaps could you have a look please ? It will avoid that we have to override all the method when using SSR

[olivermrbl]

@oliverheaps could you have a look please ? It will avoid that we have to override all the method when using SSR

Would it make sense to use Record<string, unknown> instead of object as mentioned in #1032? @adrien2p

[adrien2p]

Yes if it is accepted I can integrate it in that PR 🤘
But it will probably be Record<string, any> for the resources

[adrien2p]

@olivermrbl it's ready when you are ready 👍

[olivermrbl]

@olivermrbl it's ready when you are ready 👍

Great! I'll be doing some testing tomorrow and then we can wrap it up 👍

[adrien2p]

Should work since this is the changes Ive made for my admin