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

Documentation Update #4468

Merged
merged 6 commits into from
May 7, 2021
Merged

Conversation

StefanFl
Copy link
Contributor

@StefanFl StefanFl commented May 7, 2021

With this PR the documentation gets a much better flow. You can see how it will look like at https://stefanfl.github.io/django-DefectDojo/.

Apart from some reorganisation and lots of details there are some noteable changes:

  • About DefectDojo and Models have been merged. They were very similar. The new page is Core Data Classes
  • All the attribute descriptions in Features have been removed. I think the field names are self-explaining.
  • I didn't see the value of Usage examples, so it has been removed. The section about Risk Acceptance has been moved to the Features.
  • Please review the new Architecture page.

I am not completely happy about the Features page. DefectDojo has a lot more features, that are not mentioned here. And is everything still up-to-date? Maybe we find volunteers for that.

Copy link
Contributor

@damiencarol damiencarol left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not 100% in all the modifications. many changes are good but removing features could be discussed

Comment on lines 21 to 24
## RabbitMQ

The application server sends tasks to a [RabbitMQ](https://www.rabbitmq.com/)
message queue for asynchronous execution.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@StefanFl the architecture page is not accurate. DefectDojo is not strongly linked to RabbitMQ.
DefectDojo rely on Celery. Celery can leverage RabbitMQ for message passing but this can be replaced by Redis or another system.
I would prefer to have something more high level in this page. If you really want to talk about implementation, you can redirect users to https://docs.celeryproject.org/en/stable/getting-started/brokers/index.html#broker-overview

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I agree with @damiencarol, we should have the term broker maybe instead, and list rabbitMQ as being possibly the default for us.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Possibly some double-edges arrows in some places.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, having a Message Broker in the diagram, with a reference to RabbitMQ and Redis in the text is much better.

And can do some double-edge arrows as well.

@damiencarol
Copy link
Contributor

@StefanFl made some comment but accepting as documentation can be improved/modified all the time.

@@ -0,0 +1,77 @@
---
title: "Core data classes"
description: "DefectDojo is based on a model that allows the ultimate flexibility in your test tracking needs."
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure about that "ultimate flexibility" yet, maybe we could reword it :-)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copied it from somewhere but maybe it's really a bit much. Will reword it.

Copy link
Contributor

@madchap madchap left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Made some small comments, compared side-by-side, lgtm :) Thanks for taking care of this @StefanFl - let us know when you're ready to merge, and maybe other mods will want to look as well.

@madchap madchap requested a review from a team May 7, 2021 15:50
@StefanFl
Copy link
Contributor Author

StefanFl commented May 7, 2021

The last commit fixed the problem with the wrong link, @valentijnscholten pointed it out on Slack. Unfortunately we now get some links on the top right for editing the page and submitting issues. See https://stefanfl.github.io/django-DefectDojo/integrations/import/ for an example. The other option would have been no links to GitHub but a last modified date of 1.1.0000 or so.

For me this PR would be ready to merge

@valentijnscholten valentijnscholten merged commit 5488c93 into DefectDojo:dev May 7, 2021
@valentijnscholten
Copy link
Member

Thanks again.

@StefanFl StefanFl deleted the documentation branch May 7, 2021 17:41
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.

4 participants