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

[FEATURE] Generate sitemaps for different domains (e.g. web and mobil) #58

Closed

Conversation

felixsemmler
Copy link
Contributor

This feature is related to #34. I adapted it to version 2.x of typo3-dd_googlesitemap and fixed the things you mentioned (unescaped HTML (possible XSS) and unescaped submitted data is passed to the database (possible SQL injection)).

@vdespa
Copy link

vdespa commented Sep 17, 2015

@dmitryd This is a nice addition to the current functionality. Would it be possible to merge the PR into your code?

@dmitryd
Copy link
Owner

dmitryd commented Sep 17, 2015

I'll need time to look at it. Currently there is a lot of load at work and mot much time for free exts. But as soon as I got spare minutes, I will definitely look at it!

@dmitryd
Copy link
Owner

dmitryd commented Nov 16, 2015

A change will be needed: multiple domain records should be selectable.

One problem to solve: dev/stage/live handling with domain records.

@dmitryd
Copy link
Owner

dmitryd commented Apr 4, 2016

I thought about it and I propose to make a separate extension from it. The reason is that it will break existing installations. Now when people update extensions with composer, they can suddenly loose their existing configurations.

The best way would be to have a separate scheduler task for this in a separate extension.

@felixsemmler felixsemmler deleted the feature/select-domain-record branch November 3, 2017 08:59
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

Successfully merging this pull request may close these issues.

4 participants