Helt färsk på detta med url-omskrivning. Vad kan bli bättre i min .htaccess?

[weraw]

Ja som rubriken säger helt färsk på ämnet med url-omskrivning. Min .htaccess fil fungerar som tänkt men kan den optimeras så den blir bättre/sämre?

En annan konstig grej är att min server leverantör säger att mitt konto drar 8-10GB minne helt plötsligt ingen vet riktigt varför?

Min .htaccess

Options +FollowSymlinks
RewriteEngine on

#Nyheter,erbjudande, populart mm ersätter www.doman.se/?page=nyhter
	RewriteRule ^page/([^/]*)$ /?page=$1 [L]

# Clean up search URL <- detta kommer från en guide på nätet jag inte hittar just nu
RewriteRule ^sok/([^/\.]+)$ index.php?page=sok&sok=$1 [L]

#Skriv om adressen för produkter med kategori
	RewriteRule ^([^/]*)/([^/]*)/([^/]*)$ /?page=produkt&categori=$1&namn=$2&pid=$3 [L]

#Om kategori saknas kör denna
	RewriteRule ^([^/]*)/([^/]*)$ /?page=produkt&namn=$1&pid=$2 [L]

#Skriv om adressen för produkter
	RewriteRule ^([^/]*)/([^/]*)/([^/]*)/([^/]*)$ /?page=cat&namn=$1&cid=$2&box_open_id=$3&s=$4 [L]

[Jim Westergren]

Att din sajt plötsligt slukar så mycket minne bör inte ha att göra med din mod_rewrite. Man kan se med kommandot top på servern vad som drar minne och vilka processer det är. Kanske är det fel i din PHP-kod som orsakar oändliga loopar?

I din mod_rewrite ovan använder du mycket * vilket betyder att du accepterer allt som står för en omskrivning. Då måste du givetvis i din PHP blockera för SQL-injections, XSS och sådant. Om cid och box_open_id bara kan vara siffror så acceptera bara siffror till ex.

[weraw]

Hej Jim

Tack för snabbt svar. Jag validerar allt i mina filer. Är det tänkt att det ska vara en numretisk variabel så kollar jag så den är det tex. Likadant gäller de övriga variablarna.

Generatorn som jag använt online för att skapa url omskrivningarna sätter ett [L] i slutet på varje regel. Har läst lite vad det betyder etc.

Är det meningen att man ska ha någon if sats som tex kollar..Om kategori inte finns med i url använd denna regeln annars använd denna?

Hoppas det var det ända du hade påpeka

[Jim Westergren]

Du kan ju testa att lägga detta ovanför dina regler:

RewriteCond %{REQUEST_FILENAME} !-f
RewriteCond %{REQUEST_FILENAME} !-d

Då tillämpas reglerna endast på de requests som inte går till filer eller mappar. Det kan faktiskt hjälpa med CPU och kanske minne i vissa fall. Speciellt i de fallen där det är krävande att visa upp en 404 sida, som till exempel WordPress.

[weraw]

Hej

ska det läggas ovanför varje regel tex så här:

RewriteCond %{REQUEST_FILENAME} !-f
RewriteCond %{REQUEST_FILENAME} !-d
#Nyheter,erbjudande, populart mm
	RewriteRule ^page/([^/]*)$ /?page=$1 [L]

RewriteCond %{REQUEST_FILENAME} !-f
RewriteCond %{REQUEST_FILENAME} !-d	
# Clean up search URL
RewriteRule ^sok/([^/\.]+)$ index.php?page=sok&sok=$1 [L]

RewriteCond %{REQUEST_FILENAME} !-f
RewriteCond %{REQUEST_FILENAME} !-d
#Skriv om adressen för produkter med kategori
	RewriteRule ^([^/]*)/([^/]*)/([^/]*)$ /?page=produkt&categori=$1&namn=$2&pid=$3 [L]

RewriteCond %{REQUEST_FILENAME} !-f
RewriteCond %{REQUEST_FILENAME} !-d
#Om kategori saknas kör denna
	RewriteRule ^([^/]*)/([^/]*)$ /?page=produkt&namn=$1&pid=$2 [L]

RewriteCond %{REQUEST_FILENAME} !-f
RewriteCond %{REQUEST_FILENAME} !-d
#Skriv om adressen för produkter
	RewriteRule ^([^/]*)/([^/]*)/([^/]*)/([^/]*)$ /?page=cat&namn=$1&cid=$2&box_open_id=$3&s=$4 [L]

[welander]

RewriteRule ^page/([^/]*)$ /?page=$1 [L]

Den där tyckte jag inte var så snygg, speciellt inte om det är en svensk sida..

[weraw]

Hej Welander

Okej

Varför är det inte snyggt?
Funktionellt är det iaf.

Ja det är en svensk sida.
Åter varför och vad kan det förbättras till?

[Jim Westergren]

Nej, du ska bara ha dessa villkoren en gång överst nedanför "RewriteEngine on". Då tillämpas villkoren på alla regler nedanför tills nya villkor skulle dyka upp.

Jag tror Welander menade att det är konstigt att ha "page" som är ett engelskt ord på en svensk webbplats.

[weraw]

Ahh okej, tack jim...

[weraw]

Hej

När jag är inne med internet explorer så kan jag sitta och bläddra mellan olika produkter och helt plötsligt så laddas inte css filerna.

Adressen till css filerna blir helt plötsligt domän/kategori/produkt-namn/12312/theme/css/style.css

Den borde vara domän/theme/css/style.css

Hur kan det hända att man kan blädra bland 10-20 produkter utan problem sen blir det så helt plötsligt?

Upplerver detta endast med internet explorer 8

kan på peka att detta finns med i min index fil

<base href="http://xxxxxxxx.se/" /><!--[if IE]></base><![endif]-->

där xxxxxxxx är min domän namn

Här är .htaccess filen som är den ända och ligger i roten

#Start rewrite engine
	RewriteEngine on	
	Options +FollowSymlinks
	RewriteBase /

# Ignonera dessa mappar för att css å js ska fungera.
	RewriteRule ^(theme|css|images|js|lytebox_v5.2|jcart|info_texter|admin|webim)(/.*)?$ - [L]	

#Tips från Jim	
	RewriteCond %{REQUEST_FILENAME} !-f
	RewriteCond %{REQUEST_FILENAME} !-d
	
#Ta bort www i början av url
#Där xxxx är min domän namn :)
	RewriteCond %{HTTP_HOST} !^xxxxx.se$ [NC]
	RewriteRule ^(.*)$ http://xxxx.se/$1 [L,R=301]


## Skriv om adressen för ##

# Nyheter,Erbjudande, populärt mm
	RewriteRule ^page/([^/]*)$ /?page=$1 [L]

# Sök resultaten
	RewriteRule ^sok/([^/\.]+)$ index.php?page=sok&sok=$1 [L]
	
# Produkter med kategori
	RewriteRule ^([^/]*)/([^/]*)/([^/]*)$ /?page=produkt&categori=$1&namn=$2&pid=$3 [L]

#Om kategori saknas kör denna
	RewriteRule ^([^/]*)/([^/]*)$ /?page=produkt&namn=$1&pid=$2 [L]

#Skriv om adressen för produkter
	RewriteRule ^([^/]*)/([^/]*)/([^/]*)/([^/]*)$ /?page=cat&namn=$1&cid=$2&box_open_id=$3&s=$4 [L]

[weraw]

Hej

Har suttit och läst lite och fått till det men inte helt 100% säker på att det är den mest optimala sättet ur seo synpunkt.
Om jag lägger till [NC,R=301,L] i slutet på 2 av mina regler så fungerar css etc i internet explorer och övriga läsare.
Men vad exakt gör [NC,R=301,L] och ska jag ha det? eller finns det något annat sätt?
Det tråkiga med detta är att adressen inte längre blir enligt seo standard!
går man in på www.domän.se/mjolk-4-liter/1002
så ändras adressen till
http://www.domän.se/?page=produkt&na...liter&pid=1002

Reglerna som är berörda av ändingen

#Om kategori saknas kör denna
	RewriteRule ^([^/]*)/([^/]*)$ /?page=produkt&namn=$1&pid=$2 [NC,R=301,L]

# Produkter med kategori
	RewriteRule ^([^/]*)/([^/]*)/([^/]*)$ /?page=produkt&categori=$1&namn=$2&pid=$3 [NC,R=301,L]

[Jim Westergren]

r=301 betyder ju att göra 301 redirect. Det ska du ju inte ha på dina regler. Du verkar även ha en del annat fel nu. Anlita någon som kan hjälpa dig med detta. Tyvärr är jag uppbokad.

[Tobias Nyholm]

Hej

Har suttit och läst lite och fått till det men inte helt 100% säker på att det är den mest optimala sättet ur seo synpunkt.
Om jag lägger till [NC,R=301,L] i slutet på 2 av mina regler så fungerar css etc i internet explorer och övriga läsare.
Men vad exakt gör [NC,R=301,L] och ska jag ha det? eller finns det något annat sätt?

Jag läste igenom tråden lite. NC är No Case. R=301 är redirect av typen 301 (Som jim säger), L är att om denna regel är sann så sluta behandla fler regler.
Det är svårt att ge en exakt förklaring varför endast internet explorer strular utan att ha fått en bättre överblick över din källkod. Men om du har fått till det så att det fungerar så är det bra.

[weraw]

Hej

Japp har fått till det så det fungerar konstigt nog så var det något " som skrevs ut i produkt beskrivningen och då visades inte något i IE efter att fixat till det så fungerar allt som det ska...Underbara google placeringar för oss just nu