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

markdownify does not behave conrrectly - 2 #27

Open
bwl21 opened this issue Mar 10, 2019 · 0 comments
Open

markdownify does not behave conrrectly - 2 #27

bwl21 opened this issue Mar 10, 2019 · 0 comments

Comments

@bwl21
Copy link

bwl21 commented Mar 10, 2019

  • it merges the header of first level with the adjacent paragraph
  • it converts it all to a block quote

as an example to indicate what happens on my project:

= Anforderung =

Im Prinzip gehe ich im meinem Bereich auf „Mitarbeiter (mit E-Mail) hinzufügen“ und gebe dort nur ein Buchstaben ein. Daraufhin tauchen rechts dann alle Mitarbeiter auf, bei denen der Buchstabe mit vorkommt. Dies funktioniert mit allen Buchstaben. Es werden jedoch auch alle weiteren Infos der Mitarbeiter wie Adresse, Telefonnummer usw. angezeigt.

Natürlich können die Daten nur von „Bereichsleitern“ und nicht von jedermann eingesehen werden, wollte die Info trotzdem an euch weitergeben.

= Lösungsansatz =

das ist natürlich eine Sicherheitslücke
Lösung:
1.  wenn mehr als drei treffer sind, dann wird nichts angezeigt, sondern der Hinweis, dass die Suchkriterien verfeinert werden müssen.
2. Wir zeigen die Telefonnummern nicht mehr an.
3. es müssen mindestes drei buchstaben im Suchkriterium stehen.


= Aufgaben =

Lösung 1 implemeniteren

= Testen =

auf madb1 in irgendeinen Bereich gehen, dort bei email den buchstaben "t" eingeben,

== erwartetes Ergebenis ==

es kommt die Meldung "Zu Viele Mitarbeiter"

converts to

> 
> # AnforderungIm Prinzip gehe ich im meinem Bereich auf „Mitarbeiter (mit E-Mail) hinzufügen“ und gebe dort nur ein Buchstaben ein. Daraufhin tauchen rechts dann alle Mitarbeiter auf, bei denen der Buchstabe mit vorkommt. Dies funktioniert mit allen Buchstaben. Es werden jedoch auch alle weiteren Infos der Mitarbeiter wie Adresse, Telefonnummer usw. angezeigt.
> 
> Natürlich können die Daten nur von „Bereichsleitern“ und nicht von jedermann eingesehen werden, wollte die Info trotzdem an euch weitergeben.
> 
> # Lösungsansatzdas ist natürlich eine Sicherheitslücke
> Lösung:
> 1.  wenn mehr als drei treffer sind, dann wird nichts angezeigt, sondern der Hinweis, dass die Suchkriterien verfeinert werden müssen.
> 2. Wir zeigen die Telefonnummern nicht mehr an.
> 3. es müssen mindestes drei buchstaben im Suchkriterium stehen.
> 
> 
> # AufgabenLösung 1 implemeniteren
> 
> # Testenauf madb1 in irgendeinen Bereich gehen, dort bei email den buchstaben "t" eingeben,
> 
> ## erwartetes Ergebenis
> 
> es kommt die Meldung "Zu Viele Mitarbeiter"
> 
>   
bwl21 pushed a commit to bwl21/trac-hub that referenced this issue Mar 12, 2019
bwl21 pushed a commit to bwl21/trac-hub that referenced this issue Mar 22, 2019
There are too many changes to open a PR. nevertheless this also covers
mavam#30
mavam#28
mavam#27
mavam#26
ronaldtse added a commit to riboseinc/trac-hub that referenced this issue Jul 4, 2021
* bwl21-master:
  updated readme with hint to gitlab
  This is the wrap-up of my migration done based on tram-hub
  optimized ticket opening
  now handle attachments
  .gitignore .idea
  improve layout of transferred tickets - separate labels from badges
  export trac states to info
  improve logging and feedback to the user
  improve mapping of changes references
  improve mapping of assignee
  fixed milestone due dates
  replace full ticket urls by appropriate ticket links
  fix handling of revmap
  now handle milestones, some refinement of ticket layout
  see mavam#26 created a docker file to run trachub
  mavam#27 adapted markdownify to process header 1 correctly and not place ticket content in a quote
  see mavam#28 keep github and trac numbers in sync even if trac tickets id are not consecutive
  see mavam#30 implemented info  in ruby
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant