-
Notifications
You must be signed in to change notification settings - Fork 391
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
Update feature for Kong 0.10.x #117
Conversation
lesslucifer
commented
Aug 3, 2017
- Upstreams & target management
- SNI & Certificate management
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the PR which is adding support to many features brought with Kong 0.10, but as is it is still very buggy and not ready to be merged. I've been to really test it because I was unable to create certificates, edit SNIs, ...
</p> | ||
|
||
<p class="white-text"> | ||
... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you please elaborate on that?
</p> | ||
|
||
<p class="white-text"> | ||
... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you please elaborate on that?
</p> | ||
|
||
<p class="white-text"> | ||
... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you please elaborate on that?
</p> | ||
|
||
<p class="white-text"> | ||
... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you please elaborate on that?
<a class="btn-floating waves-effect waves-light" href="#/upstreams/{{upstream.id}}"> | ||
<i class="material-icons">mode_edit</i> | ||
</a> | ||
<a class="btn-floating waves-effect waves-light blue modal-trigger" href="#/upstreams/{{upstream.id}}/targets"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
When clicking here, I get a "Not found error"
... | ||
</p> | ||
<p class="white-text"> | ||
Checkout <a href="https://getkong.org/docs/admin-api/#add-upstream" target="_blank">Kong documentation</a> for the meaning of these parameters. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This page doesn't exist on getkong.org
}] | ||
} | ||
}) | ||
.when('/certificates', { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
When browsing this page, I got a blank page.
<app-field-error error="error.name"></app-field-error> | ||
</div> | ||
<div class="input-field col s12"> | ||
<input id="ssl_certificate_id" ng-model="sni.ssl_certificate_id" type="text" class="validate" ng-class="{invalid: error.ssl_certificate_id}"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't this field be required?
<td>{{sni.name}}</td> | ||
<td>{{sni.created_at | date}}</td> | ||
<td class="right"> | ||
<a class="btn-floating waves-effect waves-light" href="#/snis/{{sni.id}}"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
After creating an SNI, clicking this link would bring me back to /snis
. So I'm unable to edit an SNI.
<i class="material-icons">add</i> | ||
</a> | ||
|
||
<h3 class="header">snis</h3> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"SNIs", with uppercase?
* change targets get PATH
Certificates and SNIs support brought by #127 |