Trisha Anand
7a658ce5ea
App viewer user in an organization should have read:organizations permission.
2020-06-10 15:55:29 +05:30
Trisha Anand
491afc3599
In case of global datasource, no need to check for organizationId during action create.
2020-06-10 09:08:01 +00:00
Trisha Anand
a89355a5ec
Fixes bugs : 1. read:datasource permission should not be applied to application; 2. A user shouldn't be added to an organization if the user already exists in the organization.
2020-06-10 06:22:16 +00:00
Arpit Mohan
3cf8ff008f
Hacky fix for the postgres plugin to work. Including the dependency in the appsmith-server/pom.xml file instead of the plugin
2020-06-10 10:46:09 +05:30
Trisha Anand
ac06cc9668
1. user permissions during create/update of actions and datasources would now be set.
...
2. During update, policies are set to null in the update object to ensure that the policies are not overwritten to empty set.
2020-06-09 20:25:13 +05:30
Arpit Mohan
bbd33c2968
Removing the query field in ActionConfiguration and using String body as the field to store the query
...
This is to simplify the query pane on the frontend client. The client doesn't need to have separate interfaces for sql and non-sql plugins. All queries will be sent to the server in the form of a String that is parsed in different formats based on the plugin.
Also adding test cases for PostgresPlugin. Used TestContainers to simulate the postgres db in Docker inside the Java test itself. Very useful.
2020-06-09 12:12:27 +00:00
Arpit Mohan
99e7519550
Handling form data via the field bodyFormData in ActionConfiguration
...
This is because the client sends us the data for form-data in the form of a list of Property fields with keys and values. This is consistent with how the client stores & renders headers & query parameters as well.
For MediaType application/x-www-form-urlencoded we will use the bodyFormData field.
2020-06-09 05:48:16 +00:00
Trisha Anand
061577d2c9
Bug : When the user tries to invite without selecting the role user, this leads to null pointer exception.
...
Fix : A user CAN NOT be invited to an organization without specifying the role.
2020-06-08 14:56:12 +00:00
Trisha Anand
a888d54059
Changed the get all actions function to fix the bug where find by Example is not working. Instead the same has now been implemented using criteria.
2020-06-08 12:05:49 +00:00
Trisha Anand
9ac7e817cc
Merge branch 'feature/acl-remove-getCurrentOrganization' into 'feature/acl-spring-object'
...
Removed the usage of get current organization id. Added organization id to a few api calls.
See merge request theappsmith/internal-tools-server!367
2020-06-08 06:45:04 +00:00
Trisha Anand
a31796a216
Removed the usage of get current organization id. Requires addition of organization id to a few api calls.
2020-06-08 06:45:04 +00:00
Trisha Anand
f9d63f9881
Adding a 3P Api to a page now also expects the client to send the organization id.
2020-06-05 16:14:15 +05:30
Trisha Anand
87aafd6cf4
Adding test case for testDatasource
...
Refactoring the testDatasource code by moving the business logic to the service layer.
2020-06-05 10:05:23 +00:00
Arpit Mohan
1a68b7f561
Fixing the DSL Json for default layouts in new pages.
2020-06-04 16:13:47 +05:30
Trisha Anand
9e1742c289
Add OrgId to the output of the api used to return all the page ids and names given an application (id or name)
2020-06-04 08:08:15 +00:00
Arpit Mohan
1ac69bab50
Modifying the default page name to Page1
...
Also creating a default empty layout when a new page is created. This ensures that the client can show the default message on new page creations.
2020-06-04 13:05:34 +05:30
Trisha Anand
f8cf87e3c8
Invite existing user to an organization works. Updated the test case for adding user to an organization as well to use the new API.
2020-06-01 17:39:27 +00:00
Trisha Anand
f1e1e6959a
Bug : New users aren't able to read their own user object and hence homepage doesnt load.
...
Fix : Added lateral permissions for user on create.
2020-05-29 05:28:30 +00:00
Trisha Anand
d6670c70cc
Bug Root Cause : New users have no applications inside the organizations(s). In this case, get all applications does not return back organizationApplications object.
...
Fix : In getAllApplications, instead of iterating over collections of applications which could be empty, we iterate over organizations where we are guaranteed to have atleast one organization.
2020-05-28 13:31:24 +05:30
Trisha Anand
6475be63d1
Baseline code added for cascading the org level permissions to applications, pages and eventually actions.
2020-05-27 14:36:57 +00:00
Arpit Mohan
d20e92a587
Modifying the Action object to include an array of Property for dynamicBindingPathList
2020-05-27 13:16:38 +05:30
Arpit Mohan
9f82bde92c
Adding statusCode from AppsmithPluginErrors when plugins error out
...
This ensures that we can always display a status code on the client.
2020-05-26 11:50:09 +00:00
Trisha Anand
59a83d414b
Added description to roles for organization appsmith roles.
2020-05-26 10:49:42 +05:30
Arpit Mohan
a569156029
Merge branch 'hotfix/action-error' into 'release'
...
Creating the actionExecutionResult object for error scenarios as well.
See merge request theappsmith/internal-tools-server!354
2020-05-23 07:23:13 +00:00
Arpit Mohan
e6b7ffca9c
Creating the actionExecutionResult object for error scenarios as well.
...
This ensures that we can populate the request fields for all action executions
2020-05-23 12:49:01 +05:30
Trisha Anand
0c9c95bc9a
Fixed the findAll repository function which was leading to get all actions via application id to throw an internal server error.
2020-05-22 15:56:09 +05:30
Trisha Anand
d98fce3d09
WIP: adding helper functions for adding/removing roles from an organization. Adding role updates the organization right now. Removing role -> added a helper.
...
TODO : finish the above code flows + cascade these to the children objects of the organization -> Applications/Pages/Actions.
2020-05-21 15:05:43 +00:00
Shrikant Kandula
7037d99cfa
Use sequences for numbering new datasources.
2020-05-20 11:21:32 +00:00
Trisha Anand
e95b680169
create application now expects organization id. Fixed the test cases which broke because of this basic change in the way applications are created.
2020-05-19 17:09:51 +00:00
Shrikant Kandula
b13a7a5df3
Fix incorrect call to updatedAt.
2020-05-19 19:25:32 +05:30
Shrikant Kandula
329587f5ff
Fix NPE when using datasources without updatedAt.
2020-05-19 13:48:58 +00:00
Arpit Mohan
14c1778645
Merge branch 'bug/curl-auto-add-protocol' into 'release'
...
Automatically add protocol to URL provided to cURL command.
Notion ref: <https://www.notion.so/appsmith/CURL-import-for-incomplete-but-valid-URLs-like-www-google-com-leads-to-the-URL-not-getting-read-Th-9191456cb1b24cd68b92072c1a6f3ab6 >.
See merge request theappsmith/internal-tools-server!343
2020-05-19 04:23:59 +00:00
Shrikant Kandula
db0532941d
Automatically add protocol to URL provided to cURL command.
2020-05-19 04:23:58 +00:00
Arpit Mohan
85ea6563db
Merge branch 'hotfix/server-use-forward-headers' into 'release'
...
Adding the ForwardedHeaderTransform bean to enable spring to parse...
Adding the ForwardedHeaderTransform bean to enable spring to parse X-Forwarded-* headers from Nginx proxy
See merge request theappsmith/internal-tools-server!344
2020-05-19 03:56:40 +00:00
Arpit Mohan
158f5c8203
Adding the ForwardedHeaderTransform bean to enable spring to parse X-Forwarded-* headers from Nginx proxy
2020-05-19 09:22:33 +05:30
Shrikant Kandula
bffd68b1a9
Merge branch 'bug/postgres-empty-result' into 'release'
...
Don't fail after executing a non-SELECT query on Postgres.
See merge request theappsmith/internal-tools-server!341
2020-05-18 13:42:24 +00:00
Shrikant Kandula
c4d3d535a1
Rename isExecuteOnLoad to executeOnLoad.
...
The `is` prefix apparently makes Spring unhappy.
2020-05-18 12:13:54 +00:00
Shrikant Kandula
c18c344f4b
Don't fail after executing a non-SELECT query on Postgres.
2020-05-15 18:13:36 +05:30
Shrikant Kandula
180002a984
Handle errors when destroying stale connections.
...
This happens when the stale connection is an invalid connection object,
like when if it's created with invalid credentials etc.
2020-05-15 03:26:35 +00:00
Arpit Mohan
4fa254a449
Adding error log message whenever an action execution fails because of invalid datasource
2020-05-14 15:26:05 +05:30
Arpit Mohan
a9580effa7
Using the last value of duplicate headers as that is the behaviour displayed by webclient and other clients as well.
2020-05-14 08:49:06 +05:30
Arpit Mohan
71d8812feb
Fixing bug where duplicate headers were being set in the action execution result
...
Also removing empty headers from being sent to the action
2020-05-14 08:11:24 +05:30
Arpit Mohan
fb9e9fbf4c
Request body and header in Action Execution Response
2020-05-14 02:08:46 +00:00
Trisha Anand
5829a92998
Merge branch 'release' into feature/acl-spring-object
...
# Conflicts:
# appsmith-server/src/main/java/com/appsmith/server/services/ActionServiceImpl.java
2020-05-13 23:37:31 +05:30
Trisha Anand
9a8bf9dc89
In case the user is not signed in, return a 401 so that the user can be redirected to login by the frontend.
2020-05-13 18:00:03 +00:00
Shrikant Kandula
9c48f42990
Rename ActionExecutionResult.shouldCacheResponse -> isExecutionSuccess.
2020-05-12 18:25:49 +05:30
Arpit Mohan
d2d0805146
Merge branch 'feat/use-unique-name-on-post-datasource' into 'release'
...
Automatically use a unique name for creating datasources without name.
In the request from frontend for datasource creation, if the data
doesn't have a `name` value, we now set it to an automatically numbered
value and save with that.
See merge request theappsmith/internal-tools-server!333
2020-05-12 12:23:23 +00:00
Shrikant Kandula
b65690ec74
Automatically use a unique name for creating datasources without name.
...
In the request from frontend for datasource creation, if the data
doesn't have a `name` value, we now set it to an automatically numbered
value and save with that.
2020-05-12 16:05:29 +05:30
Nikhil Nandagopal
5dad85ff3e
Update form.json
2020-05-12 09:48:56 +00:00
Arpit Mohan
f4d79ae25b
Merge branch 'bug/curl-error-reporting' into 'release'
...
Report error when cURL command is invalid.
See merge request theappsmith/internal-tools-server!327
2020-05-12 04:12:55 +00:00
Shrikant Kandula
7f4d32e7d4
Report error when cURL command is invalid.
2020-05-12 04:12:55 +00:00
Trisha Anand
963a3388c3
In CI, buidling and packaging ACL branch with a new tag for the deployment.
2020-05-11 21:15:19 +05:30
Shrikant Kandula
f3e448019d
Auth mechanism is not added to URI in the MongoPlugin. Fixed now.
2020-05-11 12:16:09 +05:30
Trisha Anand
6cdab92bbe
Merge branch 'feature/acl-action-inheritance' into 'feature/acl-spring-object'
...
Action Permission Inheritance during create.
See merge request theappsmith/internal-tools-server!329
2020-05-08 15:39:36 +00:00
Trisha Anand
b7435e546f
Added action inheritance for permissions during create. No update permissions for action added. Fixed the existing test cases to run with the new code.
...
TODO : Add test cases for action permissions.
2020-05-08 15:39:36 +00:00
Trisha Anand
02de2ea1f6
Removing Arn because its not being used anymore.
2020-05-08 21:08:16 +05:30
Trisha Anand
f7bb87824c
Merge branch 'release' into feature/acl-spring-object
...
# Conflicts:
# appsmith-server/src/main/java/com/appsmith/server/domains/User.java
2020-05-08 20:57:51 +05:30
Trisha Anand
d7637355fe
Create Page always sets the inherited policies from the application. Update Application policy remains to be done. Leaving it as a TODO for now.
2020-05-08 16:32:46 +05:30
Shrikant Kandula
ffc4379a4d
Migration to install existing default plugins to existing organizations.
2020-05-08 11:46:13 +05:30
Shrikant Kandula
7673ee49eb
Merge branch 'chore/remove-index-annotations' into 'release'
...
Remove index annotations as they don't accurately reflect actual indexes
See merge request theappsmith/internal-tools-server!322
2020-05-08 01:58:43 +00:00
Shrikant Kandula
b8de4ef659
Remove index annotations as they don't accurately reflect actual indexes
2020-05-08 01:58:43 +00:00
Shrikant Kandula
2e9a94341b
Guard against NPE when action is null due to cURL command being invalid.
2020-05-08 01:22:02 +00:00
Trisha Anand
44ce0bbf01
Added baseservice function add and remove policies to an object. This would be used for updating permissions for an object.
2020-05-07 19:39:36 +05:30
Trisha Anand
957116409d
Resolved the review comments. Changed the structure of the DTO returned. Added the userPermissions as part of the base domain leading to this field always computed before returning any object to the user.
2020-05-07 10:49:48 +00:00
Trisha Anand
485eb29e03
Added timeout in DslActionDTO which is used to represent actions inside a layout and in turn is used for page load action execution.
2020-05-07 15:06:16 +05:30
Shrikant Kandula
3e2dea8f84
Merge branch 'bug/hide-exception-class-in-plugin-error-message' into 'release'
...
Hide Exception class details in the Plugin error message.
See merge request theappsmith/internal-tools-server!321
2020-05-05 11:08:38 +00:00
Shrikant Kandula
e99c104fe7
Hide Exception class details in the Plugin error message.
2020-05-05 16:34:09 +05:30
Nikhil Nandagopal
8f95fe22a9
Update form.json
2020-05-05 10:46:58 +00:00
Nikhil Nandagopal
c4f3ccfd8b
Update form.json
2020-05-05 10:34:02 +00:00
Shrikant Kandula
bc55baeb1f
Merge branch 'feat/add-delete-datasource-perm-existing-groups' into 'release'
...
Add `delete:datasources` permission to all existing groups.
See merge request theappsmith/internal-tools-server!320
2020-05-05 09:19:40 +00:00
Shrikant Kandula
172be7e7e5
Merge branch 'bug/missing-default-database-name-in-mongo-form' into 'release'
...
Add missing default database field in Mongo plugin form.
See merge request theappsmith/internal-tools-server!319
2020-05-05 09:06:32 +00:00
Shrikant Kandula
f7f03c5ad7
Add delete:datasources permission to all existing groups.
2020-05-05 13:52:03 +05:30
Shrikant Kandula
57dc2205e2
Add missing default database field in Mongo plugin form.
2020-05-05 13:39:03 +05:30
Arpit Mohan
77b4ef9a00
Adding a docker image tag for the master commit. This will allow us to revert to previous versions of the image (if required)
2020-05-05 12:54:46 +05:30
Trisha Anand
c43218eea6
Merge branch 'release' into feature/acl-spring-object
2020-05-01 17:05:43 +05:30
Trisha Anand
1f25aac3a5
Untested mock get applications for home page.
2020-05-01 16:56:05 +05:30
Shrikant Kandula
8ea8fb295e
Handle 500 when JSON body doesn't make up an object data type.
2020-04-30 13:32:24 +00:00
Trisha Anand
bc99ce9244
Merge branch 'release' into feature/acl-spring-object
2020-04-30 17:50:32 +05:30
Shrikant Kandula
c8e8c9136d
Check the value of Content-Type before trying to make an HTTP call.
2020-04-30 16:50:17 +05:30
Shrikant Kandula
50a8bad19e
Merge branch 'bug/restapi-test-without-port' into 'release'
...
Fix testing of REST API datasources fails when no port is set.
See merge request theappsmith/internal-tools-server!307
2020-04-30 06:02:28 +00:00
Shrikant Kandula
be7f1c0a81
Fix testing of REST API datasources fails when no port is set.
2020-04-30 06:02:28 +00:00
Shrikant Kandula
fc1b73773e
More readable plugin names.
2020-04-29 11:49:06 +00:00
Shrikant Kandula
86669fa7e4
Merge branch 'feat/updated-plugin-form-configs' into 'release'
...
Update form.json for datasource configurations.
See merge request theappsmith/internal-tools-server!314
2020-04-29 10:59:46 +00:00
Shrikant Kandula
06e85c308f
Update form.json for datasource configurations.
2020-04-29 16:25:30 +05:30
Shrikant Kandula
7dfa1bd8ca
Add deletedAt field into the unique index of datasources and pages.
2020-04-29 16:16:09 +05:30
Trisha Anand
8db71d00a9
Merge branch 'release' into feature/acl-spring-object
2020-04-29 15:49:57 +05:30
Trisha Anand
89f179397f
Create action with template id was leading to mock response. Removing this to allow for copy of actions which have been created using 3p API
2020-04-29 10:19:01 +00:00
Trisha Anand
0b496e1ecc
Untested add policies to an object code.
2020-04-29 15:03:56 +05:30
Shrikant Kandula
079e185ed2
Add allowUserDatasources to plugins to hide from create datasource UI.
2020-04-28 12:44:56 +00:00
Trisha Anand
2d1fe9d8b4
Merge branch 'release' into feature/acl-spring-object
...
# Conflicts:
# appsmith-interfaces/src/main/java/com/appsmith/external/models/SSHPrivateKey.java
# appsmith-server/src/main/java/com/appsmith/server/filters/AclFilter.java
# appsmith-server/src/main/java/com/appsmith/server/services/CurlImporterService.java
# appsmith-server/src/test/java/com/appsmith/server/services/CurlImporterServiceTest.java
2020-04-28 18:13:06 +05:30
Shrikant Kandula
23bd4e1722
Add NO_SSL option to SSL authType to turn off SSL explicitly.
2020-04-28 12:31:34 +00:00
Trisha Anand
219a594a4c
URL decoding the name before searching for it.
2020-04-28 16:31:19 +05:30
Trisha Anand
e99ede9d86
Minor comments added for TODOs
2020-04-28 16:18:28 +05:30
Shrikant Kandula
89d04f11e7
Merge branch 'bug/show-password-in-datasource-form' into 'release'
...
Show passwords in datasource forms.
See merge request theappsmith/internal-tools-server!308
2020-04-28 09:34:46 +00:00
Shrikant Kandula
b092692922
Merge branch 'bug/npe-if-opa-not-ready' into 'release'
...
Fix NPE when accessing services before OPA is ready.
See merge request theappsmith/internal-tools-server!300
2020-04-28 09:22:05 +00:00
Shrikant Kandula
6fd23f38b8
Merge branch 'bug/curl-parser' into 'release'
...
Fix cURL import bugs with new cURL command parser
See merge request theappsmith/internal-tools-server!306
2020-04-28 09:09:03 +00:00
Shrikant Kandula
1dbb98e1e8
Fix cURL import bugs with new cURL command parser
2020-04-28 09:09:03 +00:00
Shrikant Kandula
0a05036f9f
Show passwords in datasource forms.
2020-04-28 11:12:03 +05:30
Trisha Anand
82eb02b88c
Invite user to application as viewer or admin works now works in setting the correct permissions for the application. Next step is to set the correct permissions for the pages.
2020-04-27 18:19:28 +05:30
Shrikant Kandula
2adad6b3b5
Merge branch 'bug/spread-operator-in-mustache-templates' into 'release'
...
Fix page-load action not being picked up when using the spread operator.
See merge request theappsmith/internal-tools-server!299
2020-04-27 08:11:22 +00:00