OWASP-BLT / BLT

OWASP BLT is tool with the purpose of making the web a safer place. We have many facets to the project.
https://blt.owasp.org
GNU Affero General Public License v3.0
141 stars 143 forks source link

update the views image to show a chart and use the IP table for the data #2920

Closed DonnieBLT closed 6 days ago

DonnieBLT commented 1 week ago

Image

apoorvapendse commented 1 week ago

/assign

github-actions[bot] commented 1 week ago

Hello @apoorvapendse! You've been assigned to OWASP-BLT/BLT. You have 24 hours to complete a pull request. To place a bid and potentially earn some BCH, type /bid [amount in BCH] [BCH address].

apoorvapendse commented 1 week ago

@DonnieBLT how is the chart expected to look like? do you want to show the views from a pariticular IP address in the chart?

This is the chart I've done so far ⬇ Image

DonnieBLT commented 1 week ago

With relation to time

apoorvapendse commented 1 week ago

I don't understand @DonnieBLT , IP table just has a createdcolumn which I believe contains the date when the user with that ip first visited BLT Wouldn't we need some kind of log file for the IPs that accessed a particular project repo and perform an aggregate function on it, to plot the visit count against time?

DonnieBLT commented 1 week ago

I think that might be good enough. We use the creative date and then filter it by the path.

tsu-ki commented 1 week ago

@apoorvapendse just a request, can I take over this issue if you're not working on it at the moment? I didn't realise that you were already assigned to this and I went ahead to work on this issue, I believe I've done with almost everything. thank you!

DonnieBLT commented 1 week ago

@apoorvapendse ’s direction is closer to the outcome we are looking for.

apoorvapendse commented 1 week ago

@apoorvapendse just a request, can I take over this issue if you're not working on it at the moment? I didn't realise that you were already assigned to this and I went ahead to work on this issue, I believe I've done with almost everything. thank you!

Hey @tsu-ki , Thanks for checking in! @DonnieBLT and I are currently working through the expected behavior of the chart and the structure of the IP table. Since we're already on it, I’d recommend picking up another issue that might be a better use of your time. Thanks