feature(#172): added safari support

[KunalKumar-1]

Purpose of the PR*

#172 safari support

Priority*

• High: This PR needs to be merged first, before other tasks.
• Medium: This PR should be merged quickly to prevent conflicts due to common changes. (default)
• Low: This PR does not affect other tasks, so it can be merged later.

Changes*

How to check the feature*

• Open brie-extension in Xcode .
• Select the "BrieExtension (macOS)" scheme
• Press ⌘B (Build) to compile
• and click the execute button on the upper left corner.
• Load in Safari*
  Safari → Develop → Developer setting -> - [☑️] Allow unsigned extensions.

Reference

#172

[github-actions]

Hey @KunalKumar-1,\nThanks for another PR! 🚀

[KunalKumar-1]

let me know if any further changes to make .

[ionleu]

Good job so far,
@KunalKumar-1 !

I have a few questions:

• can we rename the root folder to safari-extension?
• is hmr working? If we do some changes, are these reflected? Or we'll need to re-build/convert the extension?
• can you put in a place a documentation for other devs to understand how we când continue developing/ maintaining the safari extension?

At the end,
I created a discussion topic, can you share there the entire process and/ or operations during working on this feature. See Discussions tab: Safari support topic.

[KunalKumar-1]

• can we rename the root folder to safari-extension?

Ofcourse we can change the folder name to safari-extension.
i will change it to safari-extension

[KunalKumar-1]

is hmr working? If we do some changes, are these reflected? Or we'll need to re-build/convert the extension?

hmr seems to work,
But we need to do some changes for Safari's CSP
In Info.plist to enable rooting for wss://

by adding it to info.plist

<string>
  connect-src wss://localhost:8081 'self';
  script-src 'self' 'unsafe-inline';
</string>

then xcrun and ⌘ +B

[KunalKumar-1]

can you put in a place a documentation for other devs to understand how we când continue developing/ maintaining the safari extension?

yup i will , please let me know the further details to do this.

[ionleu]

is hmr working? If we do some changes, are these reflected? Or we'll need to re-build/convert the extension?

hmr seems to work, But we need to do some changes in Safari's CSP In Info.plist to enable rooting for wss://

by adding it to info.plist

<string>
  connect-src wss://localhost:8081 'self';
  script-src 'self' 'unsafe-inline';
</string>

then xcrun and ⌘ +B

Was this added to info.plist?

Also,
is this an permanent solution?
I mean, no other devs will need to do this, since you'll add this to info.plist ?

[ionleu]

yup i will , please let me know the further details to do this.

You can put together a README.md file containing all necessary information and steps to run and develop the safari extension.

[KunalKumar-1]

yup i will , please let me know the further details to do this.

You can put together a README.md file containing all necessary information and steps to run and develop the safari extension.

looking into it 🔥