Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • May 27 08:46
    CircleCI failed: canax build (#21) in https://github.com/canax/url
    • Update configuration file based on production or not.
      (103cdb40888313d091abdfbaf3dab23ec03940af by Mikael Roos)
  • May 27 08:45

    mosbth on v2.0.0

    (compare)

  • May 27 08:45

    mosbth on master

    Update configuration file based… (compare)

  • Aug 15 2018 14:46
    CircleCI success: canax build (#20) in https://github.com/canax/url
    • Update Makefile.
      (7c88a464a93581f5c1dea27ae560abf88554bf7f by Mikael Roos)
  • Aug 15 2018 14:45

    mosbth on v2.0.0-beta.2

    (compare)

  • Aug 15 2018 14:45

    mosbth on master

    Update Makefile. (compare)

  • Aug 15 2018 14:18
    CircleCI success: canax build (#19) in https://github.com/canax/url
    • Update description in composer.json.
      (102476c45d5da887f16539d2d9d2bb0fb04ebef7 by Mikael Roos)
  • Aug 15 2018 14:18

    mosbth on v2.0.0-beta.1

    (compare)

  • Aug 15 2018 14:18

    mosbth on master

    Update description in composer.… (compare)

  • Aug 10 2018 08:02
    CircleCI success: canax build (#18) in https://github.com/canax/url
    • Add config/di
      (230b0d0f59b8c06f50c1104d27c328f110e3ac01 by Mikael Roos)
  • Aug 10 2018 08:01

    mosbth on v2.0.0-alpha.2

    (compare)

  • Aug 10 2018 08:01

    mosbth on master

    Add config/di (compare)

  • Aug 01 2018 15:00
    CircleCI success: canax build (#17) in https://github.com/canax/url
    • Prepare to go through code and remove some waist and potentially breaking backward compatibility.
      (74b599aa59f89606aceda0c68f626bac484ef256 by Mikael Roos)
  • Aug 01 2018 14:59

    mosbth on v2.0.0-alpha.1

    (compare)

  • Aug 01 2018 14:59

    mosbth on master

    Prepare to go through code and … (compare)

  • Dec 04 2017 10:44

    mosbth on v1.0.9

    (compare)

  • Dec 04 2017 10:44

    mosbth on master

    Fix: Not to remove ending slash. (compare)

  • Oct 12 2017 13:05

    mosbth on v1.0.8

    (compare)

  • Oct 12 2017 13:05

    mosbth on master

    Use dependecy anax/configure in… (compare)

  • Sep 25 2017 11:08

    mosbth on v1.0.7

    (compare)

Mikael Roos
@mosbth
Ah, det var url du hade fixat i, inte request... då ska vi se. https://github.com/litemerafrukt/url
Mikael Roos
@mosbth
Hmm, så en class istället för strängar som representerar en generell Uri tillsammans med helpers. Det är alltså tanken. Ok, en vinst är att koden blir mer lättläst i Url-klassen. Det är välbehövligt.
Anders Nygren
@litemerafrukt
Jo, det var det jag ville uppnå. Att lätter se hur url-klassen tar beslut och klistrar ihop saker.
Mikael Roos
@mosbth
Känns som det finns ett liknande behov av att städa i Request-klassen och lyfta ut lite saker. Ska kika om det finns relaterade saker där.
Anders Nygren
@litemerafrukt
:smile: Hehe. Tids nog. Sitter med kmom03 och hoppas på att klara grön inlämning. Tror att det kommer att brista med att få gröna inlämningar framöver, då kan jag kolla in Request-klassen.
Vill skriva tester för Uri-klassen också.
Mikael Roos
@mosbth
Det är metoderna Request::getCurrentUrl, init och extractRoute som kan ses över förutsatt att vi bestämmer oss för en Uri-klass. Det ger också en möjlighet att se över var, när och hur som encoding görs (vilket jag gärna vill göra).
Vill du försöka skriva klart Uri-klassen med det scoopet den har för tillfället? Inklusive testfall? Så kan vi merga sen och därefter se över hur det kan påverka Request-klassen?
Anders Nygren
@litemerafrukt
Hemskt gärna!
Mikael Roos
@mosbth
Jag har läst en del i RFC för encoding-detaljer och XSS-relaterade saker och vill gärna se till att Anax är skyddad mot sånt. Det är rätt pilligt och viktigt och kräver troligen en hel del egna textfall enbart mot säkerhetsgrejer. Kanske kan vi wrappa in en hel del av det i Uri-klassen.
Vi kan kika på det när Uri är klar.
Glöm inte dockblocks nu :)
Anders Nygren
@litemerafrukt
Nejdå ;)
Mikael Roos
@mosbth
Det blir bra uppdateringar som vi kan ha till hösten
Anders Nygren
@litemerafrukt
Låter bra. Skönt att ha ett sommarprojekt. Skulle bli väldigt tomt efter oophp annars :smile:
Väldigt kul att komma igång med opensource!
Mikael Roos
@mosbth
Nu är (snart) siktet inställt på ramverk1, då gör vi Anax på "riktigt" med fler moduler
Mikael Roos
@mosbth
Vore kul om vi blev ett par stycken som jobbade med detta, opensource "på riktigt"
Anders Nygren
@litemerafrukt
Jupp. Det känns som ett tryggt sätt att komma igång med open source. I ett projekt som faktiskt används.
Du får göra ordentlig reklam på grillcon ;)
Mikael Roos
@mosbth
Så är tanken :)
Anders Nygren
@litemerafrukt
Appropå egen modul. Bör UriBuilder vara egen modul, ingå i Url-modulen eller ligga i common?
Mikael Roos
@mosbth
Jag vill slakta Common, är rädd för att den kan bli en stor slasktratt.
Jag tror att Uri kan vara en egen modul, det blir bäst så. Den kommer användas av Url och Request. Vi behöver ha lite känsla när vi flyttar koden som är relaterad till Uri.
Vill du göra Uri-modulen så kör på, men räkna med att jag kommer blanda mig i :)
Jag kan göra så att du kan pusha till organisationen Canax, om du vill.
Anders Nygren
@litemerafrukt
Det hade varit spännande att kunna pusha till Canax ☺
Allt för att lära sig mer git och github!
Och naturligtvis räknar jag med att du blandar dig i 😉
Mikael Roos
@mosbth
Då kör vi, ta en befintlig modul och gör om till Uri. Säg till när du är klar så öppnar jag så du kan skapa repot under canax org.
Anders Nygren
@litemerafrukt
Tjipp. Nu har jag gjort en modul av Uri. Döpte klassen till UriBuilder. Vet inte om det är bättre eller inte. Såg det i revisionsfilen till Url, går att ändra tillbaka om Uri var bra.
Skrivit lite tester. Allt körs. Ovan testare, vet inte om det ska vara mer :smile:
Mikael Roos
@mosbth
Ta Uri på modulen, det blir ett bra generellt samlingsnamn på en (eller flera) klasser som hanterar Uri (builder eller tester).
Jag bjuder in dig till Canax så ser vi om vi kan fixa in så att du kan lägga repot där. Jag måste kanske fixa behörighet, men vi märker det.
Anders Nygren
@litemerafrukt
Tjipp. Sitter och pular runt med Url. Finns en del att göra. Få kompletta tester till exempel, bra övning :smile:
Såg i REVISION.md att du funderade på slugify i ett trait. Den ska inte ha en egen modul istället? Blir en pyttemodul men det finns samtidigt utvecklingspotential för en slugger. Tycker personligen att den passar sådär i Url-modulen. Samtidigt skulle det bli en modul till att installera för studenter... Bara en tanke
En annan sak jag noterade var att du gjort egen modul av canax/configure. Ska jag ta och ändra från common till den i Url?
Anders Nygren
@litemerafrukt
Och en sista fråga. Om jag gör små, korta, enkla, privata, helper funktioner i klasser, tex för att göra koden mer DRY. Ska vi ha någon konvention för var dessa ska ligga? Har hört alla varianter men att lägga dem sist i klassen tycker jag har låtit ganska bra.
Mikael Roos
@mosbth
Testerna kan säkert behövas städas, tror det blir enklare struktu om man samlar ihop tester i egna klasser, så blir det tydligt att varje klass/fil syftar till att testa vissa delar. Testklaserna blir lätt stora annars, svårt att överblicka när man underhåller dem.
Pass på slugify så länge, men du har en poäng
Byt till anax/configure, blir bra
sist i klassen låter bra, så man kan läsa koden ovan och ned och se det viktigaste först
Anders Nygren
@litemerafrukt
Ok. Har implementerat Uri-klassen i Url. Satt länge och funderade på om jag kunde refaktorera bort lite cyklomatisk komplexitet men kom inte på något bra sätt som inte skulle inbegripa större ombyggnader. Så det är egentligen en ganska liten refaktorering som ffa syftar till att använda Uri för att göra det lite klarare vad som händer när man läser koden.
Behövde ändra i setDefaultsFromConfiguration. Gjorde samtidigt så att man kan skicka med optionel configfil eller array som läses till $this->config mha ConfigurationTrait:et.
Åsså lade jag till lite tester för att komma till 100%. Testerna skulle behöva ses över mer men 100% är i alla fall en bra början :smile:
Mikael Roos
@mosbth
Jag är fortfarande efter min plan så nu är det efter midsommar som jag kikar på detta... men jag närmar mig. Dessa tre moduler, request, url och uri kommer att behöva ha lite extra säkerhetsfokus på hur vi hanterar $_SERVER, men vi får ta en titt på det senare.
Anders Nygren
@litemerafrukt
Jupp. Denna pull request ändrar inget i det säkerhetsavseendet.
Jag måste jobba lite på oophp innan jag börjar titta på request :wink: Så ingen brådska.