URL: https://github.com/freeipa/freeipa/pull/139
Title: #139: WebUI: Vault Management
pvomacka commented:
"""
@tiran Yes, rebased.
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/139#issuecomment-282060928
--
Manage your subscription for the Freeipa-devel mailing list:
URL: https://github.com/freeipa/freeipa/pull/139
Title: #139: WebUI: Vault Management
tiran commented:
"""
@MartinBasti you approved this PR a month ago but it has neither the ACK flag
nor was it merged.
@pvomacka Your work would be useful for my Custodia Vault work. Can you rebase
this PR
URL: https://github.com/freeipa/freeipa/pull/139
Title: #139: WebUI: Vault Management
MartinBasti commented:
"""
Works for me
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/139#issuecomment-273845743
--
Manage your subscription for the Freeipa-devel mailing list:
URL: https://github.com/freeipa/freeipa/pull/139
Title: #139: WebUI: Vault Management
pvomacka commented:
"""
@mbasti-rh Both bugs fixed, thank you.
Back to the difference between My User Vault and User Vault. I forgot to
mention that My User Vault shows only vaults which are created for the
URL: https://github.com/freeipa/freeipa/pull/139
Title: #139: WebUI: Vault Management
mbasti-rh commented:
"""
Server roles page is broken too, or at least it looks weird, probably server
names are missing
"""
See the full comment at
URL: https://github.com/freeipa/freeipa/pull/139
Title: #139: WebUI: Vault Management
mbasti-rh commented:
"""
NACK: DNS records page is broken
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/139#issuecomment-267106705
--
Manage your subscription for the Freeipa-devel
URL: https://github.com/freeipa/freeipa/pull/139
Title: #139: WebUI: Vault Management
mbasti-rh commented:
"""
> I understand that the difference between those two sections could not be very
> clear. If you have any idea on how to improve this feel free to put a comment
> here or open a
URL: https://github.com/freeipa/freeipa/pull/139
Title: #139: WebUI: Vault Management
mbasti-rh commented:
"""
NACK
1)
view My User Vaults/Add vault
There is no marked radio button and all fields are shown which are mutual
exclusive. A one option from radio group should be marked. It is doing
URL: https://github.com/freeipa/freeipa/pull/139
Title: #139: WebUI: Vault Management
pvomacka commented:
"""
Fixed PEP8 errors.
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/139#issuecomment-256271405
--
Manage your subscription for the Freeipa-devel mailing list:
URL: https://github.com/freeipa/freeipa/pull/139
Title: #139: WebUI: Vault Management
pvomacka commented:
"""
@mbasti-rh
2) fixed
3) I filled a ticket: https://fedorahosted.org/freeipa/ticket/6388
4) Tests added
5) Fixed
6) Fixed
7) Salt added
8) Field for public key added
9) Warning added
10)
URL: https://github.com/freeipa/freeipa/pull/139
Title: #139: WebUI: Vault Management
mbasti-rh commented:
"""
Yeah, and I forgot to write:
11)
There should be an information in webUI, that secrets can be added/retrieved to
vault only by using vault-archive and vault-retrieve from CLI
"""
URL: https://github.com/freeipa/freeipa/pull/139
Title: #139: WebUI: Vault Management
mbasti-rh commented:
"""
1)
I created shared vault, but I cannot see it in 'Shared Vaults', it is show only
in 'My Vaults'
i.e. it was created ad user vault according CLI
2)
'My Vaults' I expected that it
URL: https://github.com/freeipa/freeipa/pull/139
Title: #139: WebUI: Vault Management
pvoborni commented:
"""
For other optional UIs like CA/Trusts or DNS, Web UI checks on UI start if the
component is installed by batch command with:
```JavaScript
{method: "env", params: [[], {}]}
{method:
URL: https://github.com/freeipa/freeipa/pull/139
Title: #139: WebUI: Vault Management
mbasti-rh commented:
"""
I'm not sure if this is done on purpose, but Vault section is shown there even
I have no KRA installed in topology, and I'm getting error
```
An error has occurred (IPA Error 3000:
14 matches
Mail list logo