Skip to content

Refactoring source code due to deployment #13

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

Merged
merged 1 commit into from
Sep 11, 2018

Conversation

peter279k
Copy link
Collaborator

@peter279k peter279k commented Aug 11, 2018

This is related to issue #12 and issue #11 .
To @s3b4stian and @ace411, please take a look at that if having the available time 👍 .

Thanks.

Copy link
Collaborator Author

@peter279k peter279k left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is in the WIP to enhance the deployment guide.

@@ -0,0 +1,42 @@
# $Id$
Copy link
Collaborator Author

@peter279k peter279k Aug 11, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I try to run this server on Ubuntu 16.04.

This conf file is for the Apache file and it also override the 000-default.conf.
It defines the some hosts are as follows:

  • wildcard listens port number 443 and the original host listens port number 443 via HTTPS. They're same as listening port number 80 via HTTP.
  • The ServerName assumes that it's ldphp.tk.

SSLCertificateKeyFile /etc/pki/tls/private/data.fm.key
#SSLOptions +StdEnvVars +ExportCertData
SSLVerifyClient optional_no_ca
<IfModule mod_ssl.c>
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It presents that using the Let's Encrypt service to generate the SSL certificate files and the wildcard certificate files.

# If you need to use the `RewriteBase` directive, it should be the
# absolute physical path to the directory that contains this htaccess file.
#
RewriteBase /
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Using the correct rewrite approach to let all requests redirect the specific index.php.

@@ -36,7 +36,7 @@
<script>jQuery.noConflict();</script>
<script src="//cdnjs.cloudflare.com/ajax/libs/angular.js/1.1.5/angular.min.js"></script>
<script src="//cdnjs.cloudflare.com/ajax/libs/angular-ui/0.4.0/angular-ui.min.js"></script>
<script src="//w3.scripts.mit.edu/rdflib.js/dist/rdflib.js" type="text/javascript"></script>
<script src="/dist/rdflib.min.js" type="text/javascript"></script>
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is the rdflib.js link is broken.
It seems to build this JS library from this link by yourself.

@@ -13,7 +13,7 @@
}
// Exit if we don't have a #
if (strpos($path, '#') === false) // missing #
die("You must at least provide a # fragment. For example: #me or #public.");
echo "You must at least provide a # fragment. For example: #me or #public.";
Copy link
Collaborator Author

@peter279k peter279k Aug 11, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Replace die with echo to debug these messages.

@peter279k
Copy link
Collaborator Author

It's time to be merged and do the next PR for this repo 😄.

@peter279k peter279k merged commit fc53905 into linkeddata:master Sep 11, 2018
@peter279k peter279k deleted the issue_#12 branch May 17, 2019 11:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant