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
ContributorAuthor

Needs a CHANGELOG entry (#534).

@dan-f
Copy link
Contributor

dan-f commented Sep 1, 2017

lgtm 👍

@RubenVerborghRubenVerborgh merged commit d45d65d into dz_oidcSep 1, 2017
@RubenVerborghRubenVerborgh 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 freeto 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

@RubenVerborgh@dan-f