Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

matrix-synapse-rest-password-provider: init at unstable-2020-08-02, matrix-synapse-ldap3, matrix-synapse-pam: Add metadata #110004

Closed

Conversation

dali99
Copy link
Member

@dali99 dali99 commented Jan 19, 2021

Motivation for this change

Useful coupled with matrix-corporal

Also adds metadata to the other synapse plugins

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
  • Built on platform(s)
    • NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Determined the impact on package closure size (by running nix path-info -S before and after)
  • Ensured that relevant documentation is up to date
  • Fits CONTRIBUTING.md.

@dali99 dali99 force-pushed the add-matrix-synapse-rest-password-provider branch from ecd6c57 to f64763e Compare January 22, 2021 21:45
@dali99 dali99 force-pushed the add-matrix-synapse-rest-password-provider branch from f64763e to 26ee936 Compare January 22, 2021 21:46
@dali99 dali99 changed the title matrix-synapse-rest-password-provider: init at unstable-2020-08-02 matrix-synapse-rest-password-provider: init at unstable-2020-08-02, matrix-synapse-ldap3, matrix-synapse-pam: Add metadata Jan 22, 2021
@SuperSandro2000
Copy link
Member

ldap3 is missing a lib in inputs. Please check the others, too if they eval at all.

@dali99
Copy link
Member Author

dali99 commented Jan 24, 2021

I literally only push here because I want a cloud to sync to, none of these are tested yet at all.

This is marked as a draft for a reason.

@dali99
Copy link
Member Author

dali99 commented Jul 22, 2021

I never ended up using the rest password provider, and it's largely no longer necessary with matrix-corporal

@dali99 dali99 closed this Jul 22, 2021
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.

None yet

2 participants