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

Määritelmän github.io-sivuston resurssien latautumisen korjaaminen #23

Open
pe3 opened this issue Dec 1, 2015 · 14 comments
Open

Määritelmän github.io-sivuston resurssien latautumisen korjaaminen #23

pe3 opened this issue Dec 1, 2015 · 14 comments

Comments

@pe3
Copy link

pe3 commented Dec 1, 2015

Ainakin mulle tulee listallinen erroreita, kun menen sivustolle okffi.github.io/open-api-definition/ .

screen shot 2015-12-01 at 13 53 23

Resurssien linkit ovat väärin. Esim. modernizr.js löytyy osoitteesta http://okffi.github.io/open-api-definition/js/modernizr.js

@pe3 pe3 added the sivusto label Dec 1, 2015
@pe3
Copy link
Author

pe3 commented Dec 1, 2015

Näin siis kun menee repon etusivulla tarjottuun linkkiin: http://okffi.github.io/open-api-definition/. @apoikola vinkkasi toisen osoitteen, jonka kautta sivusto näyttää ehjältä http://avoinrajapinta.fi/ Ihmettelin tätä myös tuolla hyväksytyssä pull-requestissa.

Eli vaihdetaanko vaan repon linkki toimivaksi? Vai pitäisikö molemmat saada toimimaan. Toimimaton github.io-sivustohan edelleen on internetissä ja linkki siihen mahdollisesti levinnyt. Saako tuon toimimattoman github.io-sivuston poistettua ilman että se haittaa avoinrajapinta.fi-sivustoa?

@pe3 pe3 changed the title Määritelmän sivuston resurssien latautumisen korjaaminen Määritelmän github.io-sivuston resurssien latautumisen korjaaminen Dec 1, 2015
@d2s
Copy link
Contributor

d2s commented Dec 1, 2015

Hmm, tuota github.io -domainin alla olevaa osoitetta ei voi poistaa, koska se itse sivusto latautuu sieltä. CNAME -tiedostossa on määritetty oikea domain, mutta GitHub ei suoraan implementoi redirectiä oikeaan osoitteeseen. Tiedostojen URL -osoitteet ovat /abc -tyylisiä ihan siitä syystä koska ne toimivat silloin oikein sekä localhost -testiympäristössä että myös virallisessa julkisessa osoitteessa (eli http://avoinrajapinta.fi/ osoitteessa).

Potentiaalisen ongelman korjaaminen olisi mahdollista esim. jollakin JavaScript-pätkälle joka katsoisi että jos URL-osoite on väärän domainin alla, JavaScript tekisi uudelleenohjauksen uuteen sijaintiin.

@Mygee
Copy link
Contributor

Mygee commented Dec 6, 2015

Ping @pe3 ja @d2s. Ongelma on nyt korjattu. Määrittelin Jekylliin base URL:n.

@Mygee Mygee closed this as completed Dec 6, 2015
@d2s
Copy link
Contributor

d2s commented Dec 6, 2015

@Mygee Äskeinen muutoksesi korjasi osan ongelmista, mutta samalla sivun rakenteesta hajosi OpenGraph -tagit.

screenshot from 2015-12-06 22 11 34 okffi avoinrajapinta opengraph meta tags - duplicate url

Kyseinen kohta generoituu _includes/head.html tiedostosta, jossa on tällä hetkellä useampi URL-määritys peräkkäin.

 <meta property="og:image" content="{{ site.url }}{{ site.baseurl }}/img/{{ page.og-image }}">

Olettaisin että toinen kahdesta määrityksestä on turha. Pelkkä site.url riittää todennäköisesti.

@pe3
Copy link
Author

pe3 commented Dec 6, 2015

Hienoa. Arvelinkin, että noin täytyy pystyä tekemään.

@pe3
Copy link
Author

pe3 commented Dec 6, 2015

Olitpa hienosti tarkkana @d2s Hyvä, että muistit tarkistaa.

@d2s
Copy link
Contributor

d2s commented Dec 6, 2015

Jostakin syystä https://okffi.github.io/open-api-definition/ osoitteessa oleva sivu ei vieläkään näy oikein, vaikka sekä kuvatiedostojen että CSS-tiedostojen osoitteet ovatkin nyt jo oikein.

screenshot from 2015-12-06 22 21 06 avoinrajapinta

@pe3
Copy link
Author

pe3 commented Dec 6, 2015

Jälkimmäisessä näköjään käytössä HTTPS. HTTPS-sivu ei ehkä saa ladattua turvatonta tavaraa.

@d2s
Copy link
Contributor

d2s commented Dec 6, 2015

Kaikki linkit eivät sisällä oikeaa URL-osoitetta, joten ne osoittavat väärään paikkaan (ainakin alihakemistossa olevassa sivukopiossa).

screenshot from 2015-12-06 22 30 27 avoinrajapinta

En ole aivan varma, mutta osa ongelmasta johtuu _includes/post_detail.html -tiedostossa olevasta kohdasta.

<a href="{{ root_url }}{{ post.url }}">{{ post.title }}</a>

@pe3 pe3 reopened this Dec 6, 2015
@d2s
Copy link
Contributor

d2s commented Dec 22, 2015

Aikaisemmin mainittu URL-ongelma on korjattu jo jokin aika sitten Git commit c3e21fb muutoksessa. Tuolta osin linkkiongelma on siis korjattu, vaikka alkuperäistä ongelmaa resurssien lataamisen kanssa muutos ei korjaakaan.

screenshot from 2015-12-23 01 38 37 okffi git commit

@Mygee
Copy link
Contributor

Mygee commented Apr 16, 2016

Nyt Jekyll 3:n jälkeen ainakin osoite:
http://okffi.github.io/open-api-definition/ -> toimii moitteetta.
https://okffi.github.io/open-api-definition/ -> ei toimi, mutta en tiedä tarvitaanko tässä SSL:ää?

@d2s
Copy link
Contributor

d2s commented Apr 16, 2016

Jotain JavaScript -pohjaista redirect -viritelmää voisi (teoriassa) harkita. Se tarkistaisi selaimessa olevan URL-osoitteen, ja jos kyseessä olisi 'väärä' sijainti, ohjaisi sivun oikeaan URL-osoitteeseen. En tosin tiedä onko tuollaisen tekeminen erityisen tärkeää kenellekään.

@Mygee
Copy link
Contributor

Mygee commented Apr 16, 2016

isaacs/github#289

@d2s
Copy link
Contributor

d2s commented Apr 16, 2016

Ilmeisesti URL-osoitteissa pitäisi olla absolute path jotta eivät uudelleenohjautuisi vahingossa.

Esimerkiksi näin:

- <li><a href="/about">About</a></li>
+ <li><a href="/about/">About</a></li>

Tarkempi esimerkki Ruby-koodista löytyy toisesta Jekyll-projektista: talklittle/talklittle.github.com@9772836

En tosin tiedä auttaako tuo muille kuin perinteisille linkeille.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants