Skip to content
This repository was archived by the owner on Nov 5, 2025. It is now read-only.

Comments

[NEW] Enable require of some native modules and global functions#441

Merged
thassiov merged 5 commits intofeat/configure-source-storage-by-settingfrom
allowing-native-libs
Sep 23, 2021
Merged

[NEW] Enable require of some native modules and global functions#441
thassiov merged 5 commits intofeat/configure-source-storage-by-settingfrom
allowing-native-libs

Conversation

@thassiov
Copy link
Contributor

@thassiov thassiov commented Sep 3, 2021

What? ⛵

Allows the app to include code that requires the following modules:

  • http (requires networking permission)
  • https (requires networking permission)
  • zlib
  • util
  • os (as an empty object, only to fulfill some npm dependencies' requirements, as exposing it would lead to security problems)

Also, injects the following language features inside the app's context so developers can make use of:

  • timers (setInterval, setImmediate, etc)
  • Buffer

Why? 🤔

As we are allowing external dependencies in apps, some internal functionalities and libs are needed for the apps to function properly.

Links 🌎

PS 👀

@thassiov thassiov requested a review from d-gubert September 3, 2021 21:46
@thassiov
Copy link
Contributor Author

thassiov commented Sep 3, 2021

github-app.zip

tester app

@d-gubert d-gubert changed the title Modifications to work with github [NEW] Enable require of some native modules and global functions Sep 6, 2021
@d-gubert d-gubert changed the base branch from feat/configure-source-storage-by-setting to alpha September 6, 2021 20:49
@d-gubert d-gubert changed the base branch from alpha to feat/configure-source-storage-by-setting September 6, 2021 20:50
@thassiov thassiov marked this pull request as ready for review September 14, 2021 12:35
@thassiov thassiov merged commit 6183b3e into feat/configure-source-storage-by-setting Sep 23, 2021
@d-gubert d-gubert deleted the allowing-native-libs branch May 2, 2024 21:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants