Skip to content

Conversation

@RubenVerborgh
Copy link
Contributor

When an account is created, there's currently no folder that is publicly readable. This pull request adds /public with all permissions for the owner, and (inherited) read permissions for the public.

@RubenVerborgh
Copy link
Contributor Author

Needs a CHANGELOG entry (#534).

@dan-f
Copy link
Contributor

dan-f commented Sep 1, 2017

lgtm 👍

@RubenVerborgh RubenVerborgh merged commit d45d65d into dz_oidc Sep 1, 2017
@RubenVerborgh RubenVerborgh deleted the feature/public-folder branch September 1, 2017 20:47
RubenVerborgh added a commit that referenced this pull request Sep 3, 2017
* dz_oidc: (178 commits)
  Rename the idp option into multiuser (#570)
  Add a /public folder in new accounts (#569)
  Update Data Browser html file
  Remove debug overhead on ACL (#566)
  Fix requirement for additional verification logging in with WebID-TLS
  Add bootstrap.min.css.map to common/css/
  Add current hash to redirect. (#562)
  Disable rejectUnauthorized on the WebID-TLS endpoint. (#561)
  Serve static common/ dir relative to __dirname
  Tweak account index page phrasing
  Remove solid:inbox from template
  Add support for external WebIDs registering with username & password
  Move RS options to oidc-auth-manager initialization
  Do not check for user header in oidc test
  Log whole error in error handler
  Expand error message for unverified web id
  Bump oidc-auth-manager dep to 0.12.0
  Verify presence of test DNS entries. (#549)
  Expose WAC-Allow to browser clients.
  Remove async dependency.
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants