Infra
/
lemur
5
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

It's too expensive to attempt to load all certificates associated with a given notification. Some queries such as `default` are associated with a large number of certificates. We have little control over when these objects are loaded, but when marshalled they are lazyloaded via SQLAlachemy. If a user needs to get all the certificates associated with a certificate they should use the /notifications/<id>/certificates endpoints that support pagination. (#891)

This commit is contained in:
kevgliss 2017-08-28 17:57:39 -07:00 committed by GitHub
parent 82b43b5a9d
commit ff87c487c8
1 changed files with 0 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
lemur/notifications/schemas.py
View File

@ -26,7 +26,6 @@ class NotificationOutputSchema(LemurOutputSchema):
active = fields.Boolean() active = fields.Boolean()
options = fields.List(fields.Dict()) options = fields.List(fields.Dict())
plugin = fields.Nested(PluginOutputSchema) plugin = fields.Nested(PluginOutputSchema)
certificates = fields.Nested(AssociatedCertificateSchema, many=True, missing=[])
@post_dump @post_dump
def fill_object(self, data): def fill_object(self, data):